You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Do you think we should fix it up? change the schemaType of LongSerializer and ShortSerializer from number to integer.
I notice the interface SchemaAware is removed in 3.x , but the latest release is still 2.x . New developers may still override methods of SchemaAware. I think interface SchemaAware should be marked with @Deprecated.
The text was updated successfully, but these errors were encountered:
@luozhenyu Yes, I think you are right, these should be changed to give more specific type. This can go in 2.15.
As to SchemaAware... it might make sense to mark it deprecated; the main problem is that this will lead to lots of warnings for serializer implementation in databind package.
But other 2 types in that package are indeed deprecated so maybe it should be.
Could you please file a separate issue suggesting adding deprecation?
And I'll go ahead change schema type under this issue.
Describe the bug
The
schemaType
ofLongSerializer
andShortSerializer
should beinteger
instead ofnumber
.https://github.com/FasterXML/jackson-databind/blob/2.15/src/main/java/com/fasterxml/jackson/databind/ser/std/NumberSerializers.java#L188-L190
Version information
2.x
To Reproduce
Expected behavior
schemaType
ofLongSerializer
andShortSerializer
from number to integer.SchemaAware
is removed in 3.x , but the latest release is still 2.x . New developers may still override methods of SchemaAware. I thinkinterface SchemaAware
should be marked with@Deprecated
.The text was updated successfully, but these errors were encountered: