-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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: seata-server remove cms parameters #4434
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #4434 +/- ##
=============================================
- Coverage 48.95% 48.53% -0.43%
+ Complexity 4168 4134 -34
=============================================
Files 741 741
Lines 26521 26521
Branches 3294 3294
=============================================
- Hits 12984 12871 -113
- Misses 12129 12255 +126
+ Partials 1408 1395 -13
|
这个pr就是支持jdk17的 |
请问什么时候能合并此 PR 发布新版本以支持 |
这个PR应该是server端的启动脚本,兼容JDK17。 |
|
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
byte-buddy替换cglib
Ⅱ. Does this pull request fix one issue?
fixes #4123
fixes #4663
fixes #4703
fixes #4752
fixes #4821
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews