-
Notifications
You must be signed in to change notification settings - Fork 26.4k
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
2.5.9 Annotation Enhancement and Bugfix #1205
Conversation
to fix alibaba/dubbo apache#1120
To resolve alibaba/dubbo apache#1125
To resolve alibaba/dubbo apache#1125
multiple Config Bean To resolve alibaba/dubbo apache#1141
Codecov Report
@@ Coverage Diff @@
## 2.5.x #1205 +/- ##
==========================================
- Coverage 27.55% 27.51% -0.05%
==========================================
Files 789 790 +1
Lines 41565 41587 +22
Branches 8476 8480 +4
==========================================
- Hits 11454 11441 -13
- Misses 28061 28109 +48
+ Partials 2050 2037 -13
Continue to review full report at Codecov.
|
@mercyblitz Would you please revert the dubbo version? There are too many irrelevant changes that bothers when reviewing. |
@chickenlj Done |
* Fixed apache#1116 apache#1120 apache#1121 apache#1122 apache#1141 apache#1125 * DubboConfigConfiguration's Single and Multiple to be public classes * @DubboComponentScan is compatible with Spring @service completely * @service and @reference issues on <dubbo:annotation> * To fix an issue on <dubbo:annotation> when id attribute is absent * To change the prefixes of properties when @EnableDubboConfig binds multiple Config Bean
What is the purpose of the change
Annotation Enhancement and bugfix
Bugfix
@DubboComponentScan
is not compatible Spring @service completely @DubboComponentScan is not compatible Spring @Service completely #1120Enhancement
ReferenceBean
should be exposable for production-ready features ReferenceBean should be exposable for production-ready features #1121DubboConfigConfiguration
's Single and Multiple to be public classes DubboConfigConfiguration's Single and Multiple to be public classes #1122Break Change
Verifying this change
Run use-tests