Try to fix #4626, #4630 (start with failing tests)

[cowtowncoder]

No description provided.

[cowtowncoder]

Not sure this is going anywhere, bumping into issue of #4634, among other problems.

[cowtowncoder]

Think I might have solved the problems by only removing Record Fields:

1. After annotations merged AND
2. Only for deserialization (where we shouldn't change value)

[cowtowncoder]

@yihtserns WDYT? Looks like this change works, with your earlier fix.

[yihtserns]

@cowtowncoder Why still drop the fields, though? To avoid any other unexpected side-effects?

[yihtserns]

Oh right, dropping fields to avoid SettableBeanProperty (specifically FieldProperty) from being created for them. OK.

Yeah I guess this is a good alternative to #4627.

[cowtowncoder]

Oh right, dropping fields to avoid SettableBeanProperty (specifically FieldProperty) from being created for them. OK.

Yeah I guess this is a good alternative to #4627.

Correct, so Field is never used as mutator for Records (it fixes one specific test).