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

Jackson build time serializers could use SerializableString field names #42619

Closed
franz1981 opened this issue Aug 19, 2024 · 1 comment · Fixed by #42651
Closed

Jackson build time serializers could use SerializableString field names #42619

franz1981 opened this issue Aug 19, 2024 · 1 comment · Fixed by #42651
Labels
area/jackson Issues related to Jackson (JSON library) kind/enhancement New feature or request
Milestone

Comments

@franz1981
Copy link
Contributor

franz1981 commented Aug 19, 2024

Description

Based on https://github.com/FasterXML/jackson-core/blob/8744bd42770c9e277d995ef00fb518940efef3ef/src%2Fmain%2Fjava%2Fcom%2Ffasterxml%2Fjackson%2Fcore%2FJsonGenerator.java#L1130 and the actual implementation, moving to precomputed SerializableString(s) fields (or, even better, lazily populated, to avoid the footprint cost for entities which will never be used), injected in the build time Jackson Serializers, could improve dramatically JSON encoding.
It will basically skip all the required validations to escape names necessary to produce valid JSON data and would likely make the encoding as fast as an array copy.

@franz1981 franz1981 added the kind/enhancement New feature or request label Aug 19, 2024
@quarkus-bot quarkus-bot bot added the area/jackson Issues related to Jackson (JSON library) label Aug 19, 2024
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 19, 2024

/cc @geoand (jackson), @gsmet (jackson), @mariofusco (jackson)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/jackson Issues related to Jackson (JSON library) kind/enhancement New feature or request
Projects
None yet
2 participants