-
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
bugfix:RM delete delay registration #830
Conversation
merge from seata
merge from origin
merge from origin
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.
Didn't change completely, but I will modify it later.
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.
check TableMetaTest
done |
Codecov Report
@@ Coverage Diff @@
## develop #830 +/- ##
=============================================
+ Coverage 38.28% 38.67% +0.39%
- Complexity 1013 1026 +13
=============================================
Files 220 220
Lines 8504 8498 -6
Branches 1023 1023
=============================================
+ Hits 3256 3287 +31
+ Misses 4861 4823 -38
- Partials 387 388 +1
Continue to review full report at Codecov.
|
Ⅰ. Describe what this PR did
Delete the delayed registration of RM, and resolve the bug that the rollback retry fails after the service restarts.
Ⅱ. Does this pull request fix one issue?
#818
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews