-
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: opt the logic of SpringProxyUtils.findTargetClass(#3155) #3156
optimize: opt the logic of SpringProxyUtils.findTargetClass(#3155) #3156
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3156 +/- ##
=============================================
- Coverage 50.37% 50.26% -0.12%
+ Complexity 3116 3112 -4
=============================================
Files 594 594
Lines 19625 19591 -34
Branches 2438 2390 -48
=============================================
- Hits 9887 9847 -40
+ Misses 8744 8725 -19
- Partials 994 1019 +25 |
please standardize the title |
spring/src/main/java/io/seata/spring/util/SpringProxyUtils.java
Outdated
Show resolved
Hide resolved
…llliu/seata into optimize-spring-proxy-utils
Codecov Report
@@ Coverage Diff @@
## develop #3156 +/- ##
=============================================
- Coverage 51.11% 50.95% -0.17%
+ Complexity 3297 3289 -8
=============================================
Files 614 614
Lines 20125 20141 +16
Branches 2514 2519 +5
=============================================
- Hits 10287 10262 -25
- Misses 8806 8851 +45
+ Partials 1032 1028 -4
|
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
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
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
fixes #3155
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews