Skip to content

Change MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS default to false for 3.0 (if and when MethodHandles used) #5074

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
Open
cowtowncoder opened this issue Apr 7, 2025 · 1 comment
Open

Change MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS default to false for 3.0 (if and when MethodHandles used) #5074

cowtowncoder opened this issue Apr 7, 2025 · 1 comment
Labels
3.0-release-notes Issues relevant for 3.0 release notes. 3.0 Issue planned for initial 3.0 release performance Issue related to performance problems or enhancements

Comments

@cowtowncoder
Copy link
Member

cowtowncoder commented Apr 7, 2025 •
edited
Loading

(note: follow-up to #5046)

With change to MethodHandles it seems there should no longer be any benefit from trying to force access of public accessors (Fields, Methods). While we may consider even fully removing this feature, let's first make it disabled by default.

NOTE: once implemented, need to update

https://github.com/FasterXML/jackson-future-ideas/wiki/JSTEP-2

page.
@cowtowncoder cowtowncoder added 3.0 Issue planned for initial 3.0 release performance Issue related to performance problems or enhancements 3.0-release-notes Issues relevant for 3.0 release notes. labels Apr 7, 2025
@cowtowncoder cowtowncoder mentioned this issue Apr 7, 2025
Merged
@cowtowncoder
Copy link
Member Author

NOTE: #5046 got reverted -- this task only makes sense if and when that change gets re-merged.

@cowtowncoder cowtowncoder changed the title Change MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS default to false for 3.0 Change MapperFeature.OVERRIDE_PUBLIC_ACCESS_MODIFIERS default to false for 3.0 (if and when MethodHandles used) Apr 10, 2025
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-release-notes Issues relevant for 3.0 release notes. 3.0 Issue planned for initial 3.0 release performance Issue related to performance problems or enhancements
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cowtowncoder