Skip to content

Remove obsolete RuntimeHintsUtils #29058

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
sbrannen opened this issue Sep 2, 2022 · 3 comments
Closed

Remove obsolete RuntimeHintsUtils #29058

sbrannen opened this issue Sep 2, 2022 · 3 comments
Assignees
@sdeleuze
Labels
in: core Issues in core modules (aop, beans, core, context, expression) theme: aot An issue related to Ahead-of-time processing type: enhancement A general enhancement
Milestone
6.0.0-M6

Comments

@sbrannen
Copy link
Member

sbrannen commented Sep 2, 2022

Overview

Since Spring no longer adds the SynthesizedAnnotation interface to the JDK dynamic proxy used to synthesize an annotation, SynthesizedAnnotation and related methods in RuntimeHintsUtils have been deprecated for removal prior to 6.0.

Consequently, RuntimeHintsUtils no longer has any functionality that is not deprecated and should be removed unless we find that there are other use cases that should be supported in RuntimeHintsUtils.

Related Issues
@sbrannen sbrannen added in: core Issues in core modules (aop, beans, core, context, expression) type: enhancement A general enhancement theme: aot An issue related to Ahead-of-time processing labels Sep 2, 2022
@sbrannen sbrannen added this to the 6.0.0-M6 milestone Sep 2, 2022
@sbrannen sbrannen mentioned this issue Sep 2, 2022
Closed
@sdeleuze
Copy link
Contributor

sdeleuze commented Sep 6, 2022

Since #29083 has been merged, I guess we should keep RuntimeHintsUtils and just fix #29059.

@sdeleuze sdeleuze closed this as not planned Won't fix, can't repro, duplicate, stale Sep 6, 2022
@sdeleuze sdeleuze removed this from the 6.0.0-M6 milestone Sep 6, 2022
@sdeleuze sdeleuze added the status: declined A suggestion or change that we don't feel we should currently apply label Sep 6, 2022
@sbrannen
Copy link
Member Author

sbrannen commented Sep 6, 2022

Reopening due to 8fbd214

@sbrannen sbrannen reopened this Sep 6, 2022
@sbrannen sbrannen removed the status: declined A suggestion or change that we don't feel we should currently apply label Sep 6, 2022
@sbrannen sbrannen added this to the 6.0.0-M6 milestone Sep 6, 2022
@sdeleuze sdeleuze reopened this Sep 6, 2022
@sdeleuze
Copy link
Contributor

sdeleuze commented Sep 6, 2022

Temporarily reverted in order to let the time to portfolio projects to remove RuntimeHintsUtils usage.

@sbrannen sbrannen changed the title Consider removal of RuntimeHintsUtils Remove obsolete RuntimeHintsUtils Sep 6, 2022
This was referenced Sep 6, 2022
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sdeleuze sdeleuze

Labels
in: core Issues in core modules (aop, beans, core, context, expression) theme: aot An issue related to Ahead-of-time processing type: enhancement A general enhancement
Projects
None yet
Milestone
6.0.0-M6
Development

No branches or pull requests
2 participants
@sbrannen @sdeleuze