Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 3.3 #16144 +/- ##
============================================
+ Coverage 60.78% 60.79% +0.01%
- Complexity 11746 11754 +8
============================================
Files 1953 1953
Lines 89118 89118
Branches 13444 13444
============================================
+ Hits 54166 54181 +15
+ Misses 29382 29371 -11
+ Partials 5570 5566 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
In this mode, ensure that invoker.getInterface() (i.e., GenericService) is placed at the forefront. |
9ea4b02 to
fd5e89a
Compare
|
this pr might cause problem if the dubbo generice service references are created without annotation |
It seems that the only scenario where the interface order of AbstractProxyFactory needs to take effect for Spring is when users entirely bypass ReferenceBean and directly call referenceConfig.get() in a @bean method—which is precisely the case this PR aims to fix. |
|
It seems that scenarios without @bean won't cause issues either, because those cases are protected by the lazy proxy layer. |
What is the purpose of the change?
Close #16083
Checklist