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.

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

Remove @FunctionalInterface declaration from BeanOverrideProcessor #34259

Closed
sbrannen opened this issue Jan 14, 2025 · 0 comments
Closed

Remove @FunctionalInterface declaration from BeanOverrideProcessor #34259

sbrannen opened this issue Jan 14, 2025 · 0 comments
Assignees
@sbrannen
Labels
in: test Issues in the test module type: enhancement A general enhancement
Milestone
6.2.2

Comments

@sbrannen
Copy link
Member

A BeanOverrideProcessor is always instantiated via reflection and is never supplied as a lambda expression. In addition, we may later need to introduce additional methods in the BeanOverrideProcessor interface (which is the case for #33925, even though that new method will be a default method).

In light of that, we should remove the @FunctionalInterface declaration from BeanOverrideProcessor.
@sbrannen sbrannen added in: test Issues in the test module type: enhancement A general enhancement labels Jan 14, 2025
@sbrannen sbrannen added this to the 6.2.2 milestone Jan 14, 2025
@sbrannen sbrannen self-assigned this Jan 14, 2025
@sbrannen sbrannen changed the title Remov @FunctionalInterface declaration from BeanOverrideProcessor Remove @FunctionalInterface declaration from BeanOverrideProcessor Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sbrannen sbrannen

Labels
in: test Issues in the test module type: enhancement A general enhancement
Projects
None yet
Milestone
6.2.2
Development

No branches or pull requests
1 participant
@sbrannen