Skip to content

Conversation

@quaff
Copy link
Contributor

@quaff quaff commented Oct 14, 2024

Align with ConfigurationPropertiesBinder and ConfigurationPropertiesBindingPostProcessor

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 14, 2024
@quaff
Copy link
Contributor Author

quaff commented Oct 14, 2024

There are several places are using genericBeanDefinition(), but rootBeanDefinition() is more common in Spring Boot codebase, I would like to amend this PR if the team think it's worthy to change them also.

@quaff
Copy link
Contributor Author

quaff commented Oct 14, 2024

@snicoll
Copy link
Member

snicoll commented Oct 14, 2024

Yes, RootBeanDefinition is indeed recommended as the framework will built one internally via the merge process. So having it from the get go allows to skip that bit. There are places were we can't use it though. If you still want to amend this PR to use RBD, please let us know.

@quaff
Copy link
Contributor Author

quaff commented Oct 14, 2024

Yes, RootBeanDefinition is indeed recommended as the framework will built one internally via the merge process. So having it from the get go allows to skip that bit. There are places were we can't use it though. If you still want to amend this PR to use RBD, please let us know.

I will amend it later.

@quaff quaff force-pushed the patch-80 branch 2 times, most recently from 234e2a7 to d4035a8 Compare October 15, 2024 00:39
@quaff quaff changed the title Register BoundConfigurationProperties as RootBeanDefinition instead of GenericBeanDefinition Register beans as RootBeanDefinition instead of GenericBeanDefinition if feasible Oct 15, 2024
@quaff
Copy link
Contributor Author

quaff commented Oct 15, 2024

> Task :spring-boot-project:spring-boot-actuator:compileJava
/home/runner/work/spring-boot/spring-boot/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/web/reactive/AbstractWebFluxEndpointHandlerMapping.java:527: warning: [deprecation] check(Supplier<Authentication>,T) in AuthorityAuthorizationManager has been deprecated
				.check(this::getAuthentication, null)
				^
  where T is a type-variable:
    T extends Object declared in class AuthorityAuthorizationManager
error: warnings found and -Werror specified
1 error
1 warning

> Task :spring-boot-project:spring-boot-actuator:compileJava FAILED

It's weird, this task works on my machine.

@quaff
Copy link
Contributor Author

quaff commented Oct 15, 2024

> Task :spring-boot-project:spring-boot-actuator:compileJava
/home/runner/work/spring-boot/spring-boot/spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/web/reactive/AbstractWebFluxEndpointHandlerMapping.java:527: warning: [deprecation] check(Supplier<Authentication>,T) in AuthorityAuthorizationManager has been deprecated
				.check(this::getAuthentication, null)
				^
  where T is a type-variable:
    T extends Object declared in class AuthorityAuthorizationManager
error: warnings found and -Werror specified
1 error
1 warning

> Task :spring-boot-project:spring-boot-actuator:compileJava FAILED

It's weird, this task works on my machine.

Fixed by #42679

… if feasible

Spring Framework recommend to use RootBeanDefinition if it's pre-determined as root bean.
@snicoll snicoll added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Oct 15, 2024
@snicoll snicoll added this to the 3.4.0-RC1 milestone Oct 15, 2024
@snicoll snicoll self-assigned this Oct 15, 2024
snicoll pushed a commit that referenced this pull request Oct 15, 2024
This commit adapts code that was using GenericBeanDefinition to use
RootBeanDefinition instead. Spring Framework recommend to use
RootBeanDefinition if it's pre-determined as root bean.

See gh-42611
snicoll added a commit that referenced this pull request Oct 15, 2024
This commit adapts code that was using GenericBeanDefinition to use
RootBeanDefinition instead. Spring Framework recommend to use
RootBeanDefinition if it's pre-determined as root bean.

See gh-42611
@snicoll snicoll closed this in 389dead Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: task A general task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants