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

Remove reflective RecordFieldDiscoverer #295

Open
zml2008 opened this issue Jun 10, 2022 · 0 comments
Open

Remove reflective RecordFieldDiscoverer #295

zml2008 opened this issue Jun 10, 2022 · 0 comments
Milestone

Comments

@zml2008
Copy link
Member

zml2008 commented Jun 10, 2022

Currently, the record field discoverer has two implementations: one which accesses Class methods reflectively (from the J14/J15 era), and one in a multi-release folder which directly calls the same methods. As the J14/J15 record implementations were only preview features, they are mostly unused in production environments. Additionally, 14 is already out of support, and 15 is soon to be out of support (March 2023 for Azul's distribution), so there should not be many remaining users.

What few users there are can delay upgrading Configurate until they've been able to update to at least Java 16.

We can remove the reflection-based implementation in favor of the directly compiled implementation that we already have, to reduce required maintenance effort for the project.

@zml2008 zml2008 added this to the 4.3.0 milestone Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant