Skip to content

Change the way BeanDescription passed during deserializer construction to use Supplier #5094

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.

Sign up for GitHub

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

Jump to bottom
Closed
cowtowncoder opened this issue Apr 13, 2025 · 0 comments
Closed

Change the way BeanDescription passed during deserializer construction to use Supplier #5094

cowtowncoder opened this issue Apr 13, 2025 · 0 comments
Labels
3.0 Issue planned for initial 3.0 release
Milestone
3.0.0-rc4

Comments

@cowtowncoder
Copy link
Member

To support lazy introspection of BasicBeanDescription for common cases (or more specifically, often avoid it altogether), it makes sense to pass Supplier<BeanDescription> instead of BeanDescription.
Supplier will then just call introspection first time, and cache (memoize) it for further access.

The most obvious place that needs changing is Deserializers but code in BasicDeserializerFactory and BeanDeserializerFactory needs changes too.

And after databind changes, many/most modules as well.
@cowtowncoder cowtowncoder added the 3.0 Issue planned for initial 3.0 release label Apr 13, 2025
cowtowncoder added a commit that referenced this issue Apr 15, 2025
@cowtowncoder
Fixed #5094: add lazy construction of BeanDescription for deserializa…
0a04ff2 
…tion
@cowtowncoder cowtowncoder mentioned this issue Apr 16, 2025
Merged
cowtowncoder added a commit that referenced this issue Apr 16, 2025
@cowtowncoder
Fixed #5094: add lazy construction of BeanDescription for deserializa…
a0948db 
…tion (#5096)
cowtowncoder added a commit that referenced this issue Apr 16, 2025
@cowtowncoder
Bit of refactoring for BeanDescription.Supplier (post #5093, #5094)
4cd512d
@cowtowncoder cowtowncoder added this to the 3.0.0-rc4 milestone Apr 22, 2025
cowtowncoder pushed a commit to JooHyukKim/jackson-databind that referenced this issue Apr 25, 2025
@iifawzi
Fixing records deserialization issue (FasterXML#5094, parts of Faster…
31bfb13 
…XML#4628) with read-access when json renamed to same property name (FasterXML#5073)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.0 Issue planned for initial 3.0 release
Projects
None yet
Milestone
3.0.0-rc4
Development

No branches or pull requests
1 participant
@cowtowncoder