-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
optimize: Transfer the operation of adding configuration class from 'AutoConfiguration' to 'EnvironmentPostProcessor' #3765
optimize: Transfer the operation of adding configuration class from 'AutoConfiguration' to 'EnvironmentPostProcessor' #3765
Conversation
…AutoConfiguration' to 'EnvironmentPostProcessor'
Codecov Report
@@ Coverage Diff @@
## develop #3765 +/- ##
=============================================
- Coverage 40.37% 40.36% -0.01%
- Complexity 3070 3071 +1
=============================================
Files 685 686 +1
Lines 23194 23201 +7
Branches 2869 2868 -1
=============================================
+ Hits 9364 9366 +2
- Misses 12960 12967 +7
+ Partials 870 868 -2
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
…/spring-boot-configuration
core/src/main/java/io/seata/core/rpc/netty/AbstractNettyRemoting.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…AutoConfiguration' to 'EnvironmentPostProcessor' (apache#3765)
optimize: Transfer the operation of adding configuration class from 'AutoConfiguration' to 'EnvironmentPostProcessor'.
优化:将 添加配置类的操作 从
AutoConfiguration
转移到EnvironmentPostProcessor
中。提升该操作的优先级,避免Bean加载了,配置类却还没有添加进去,导致读取配置异常。