Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate ObjectMapper.canDeserialize()/ObjectMapper.canSerialize() in 2.18 #4570

Closed
cowtowncoder opened this issue Jun 6, 2024 · 2 comments
Labels
Milestone

Comments

@cowtowncoder
Copy link
Member

cowtowncoder commented Jun 6, 2024

(note: offshoot of discussion in #4568)

As per #1917 there is limited value in these 2 methods (and underlying machinery), and hence they are dropped from 3.0, as we cannot drop them from 2.x due to backwards-compatibility requirements.
But since the fundamental problem exists, we should mark them as @deprecated in 2.x (specifically, 2.18).

@JooHyukKim
Copy link
Member

Along with deprecating the two methods, shall we come up with alternative way to achieve the same functionality which may be added to deprecated methos' javadocs so users know what to do later on?

@cowtowncoder
Copy link
Member Author

@JooHyukKim There isn't really matching functionality, unfortunately: there is no good way (that I know of) to figure it in useful manner whether something is reliably (de)serializable; from what kind of input and so on. Calling applications should not -- for example -- try to obtain serializer or deserializer.

I am +1 on improvements to Javadocs, just not sure what to say other than "these seemed like a good idea but weren't".

@cowtowncoder cowtowncoder added this to the 2.18.0 milestone Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants