-
Notifications
You must be signed in to change notification settings - Fork 8.9k
bugfix: shentongdata xa mode should be hold the same connection #7771
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: shentongdata xa mode should be hold the same connection #7771
The head ref may contain hidden characters: "bugfix\uFF1Ashentong-xa-should-be-hold-same-connect"
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 2.x #7771 +/- ##
============================================
- Coverage 69.23% 69.19% -0.04%
- Complexity 992 994 +2
============================================
Files 1322 1322
Lines 50173 50175 +2
Branches 5931 5932 +1
============================================
- Hits 34736 34721 -15
- Misses 12523 12541 +18
+ Partials 2914 2913 -1
🚀 New features to boost your workflow:
|
|
Please add change log @dsomehan |
…d-same-connect' into bugfix:shentong-xa-should-be-hold-same-connect # Conflicts: # changes/en-us/2.x.md # changes/zh-cn/2.x.md
maple525866
left a comment
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
slievrly
left a comment
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
The jdbc implementation of shentongdata requires the same XaConnection object for the same transaction
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews