-
Notifications
You must be signed in to change notification settings - Fork 26.5k
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
fix default setting #5067
fix default setting #5067
Conversation
hi @brucelwl would you mind resolving the conflict so we can merge it |
* type definition for String class * code format
|
Codecov Report
@@ Coverage Diff @@
## master #5067 +/- ##
============================================
- Coverage 64.16% 64.15% -0.01%
Complexity 462 462
============================================
Files 769 769
Lines 33250 33250
Branches 5238 5238
============================================
- Hits 21335 21332 -3
+ Misses 9493 9492 -1
- Partials 2422 2426 +4
Continue to review full report at Codecov.
|
1、需要通过AnnotatedMethodElement 获取里面的注解信息及引用对象,但是这个类是私有的, 将他改成public,比较合理.
2、还有 DubboConfigBindingBeanPostProcessor 将这两个属性值固定死了, 无论外部bean怎么配置都会在这里改变,所以将这两个属性改成局部变量比较合理
private boolean ignoreUnknownFields = true;
private boolean ignoreInvalidFields = true;
这是2.7.x版本,请看2.6.x 版本:#4955