Skip to content

Commit e7dde74

Browse files
committed
Reduced DefaultListableBeanFactory's override logging to debug level for equivalent bean definition
Issue: SPR-7607
1 parent 314b069 commit e7dde74

File tree

1 file changed

+11
-3
lines changed

1 file changed

+11
-3
lines changed

Diff for: spring-beans/src/main/java/org/springframework/beans/factory/support/DefaultListableBeanFactory.java

+11-3
Original file line numberDiff line numberDiff line change
@@ -824,14 +824,22 @@ else if (oldBeanDefinition.getRole() < beanDefinition.getRole()) {
824824
// e.g. was ROLE_APPLICATION, now overriding with ROLE_SUPPORT or ROLE_INFRASTRUCTURE
825825
if (this.logger.isWarnEnabled()) {
826826
this.logger.warn("Overriding user-defined bean definition for bean '" + beanName +
827-
" with a framework-generated bean definition ': replacing [" +
827+
"' with a framework-generated bean definition: replacing [" +
828828
oldBeanDefinition + "] with [" + beanDefinition + "]");
829829
}
830830
}
831-
else {
831+
else if (!beanDefinition.equals(oldBeanDefinition)) {
832832
if (this.logger.isInfoEnabled()) {
833833
this.logger.info("Overriding bean definition for bean '" + beanName +
834-
"': replacing [" + oldBeanDefinition + "] with [" + beanDefinition + "]");
834+
"' with a different definition: replacing [" + oldBeanDefinition +
835+
"] with [" + beanDefinition + "]");
836+
}
837+
}
838+
else {
839+
if (this.logger.isDebugEnabled()) {
840+
this.logger.debug("Overriding bean definition for bean '" + beanName +
841+
"' with an equivalent definition: replacing [" + oldBeanDefinition +
842+
"] with [" + beanDefinition + "]");
835843
}
836844
}
837845
}

0 commit comments

Comments
 (0)