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: global session is not change to Committed in saga mode #5050
Conversation
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 PTAL
Codecov Report
@@ Coverage Diff @@
## develop #5050 +/- ##
=============================================
- Coverage 49.44% 49.42% -0.02%
+ Complexity 4124 4123 -1
=============================================
Files 733 733
Lines 26051 26051
Branches 3219 3219
=============================================
- Hits 12880 12876 -4
- Misses 11794 11800 +6
+ Partials 1377 1375 -2
|
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
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
changes/zh-cn/develop.md
Outdated
@@ -32,6 +32,7 @@ | |||
- [[#5004](https://github.com/seata/seata/pull/5004)] 修复mysql update join行数据重复的问题 | |||
- [[#5033](https://github.com/seata/seata/pull/5033)] 修复InsertOnDuplicateUpdate的SQL语句中无插入列字段导致的空指针问题 | |||
- [[#5038](https://github.com/seata/seata/pull/5038)] 修复SagaAsyncThreadPoolProperties冲突问题 | |||
- [[#5050](https://github.com/seata/seata/pull/5050)] 修复Saga模式下全局状态未正确更改成Commited |
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.
fix一下typo,Committed
changes/en-us/develop.md
Outdated
@@ -30,6 +30,7 @@ Add changes here for all PR submitted to the develop branch. | |||
- [[#5004](https://github.com/seata/seata/pull/5004)] fix duplicate image row for update join | |||
- [[#5033](https://github.com/seata/seata/pull/5033)] fix null exception when sql columns is empty for insert on duplicate | |||
- [[#5038](https://github.com/seata/seata/pull/5038)] remove @EnableConfigurationProperties({SagaAsyncThreadPoolProperties.class}) | |||
- [[#5050](https://github.com/seata/seata/pull/5050)] fix global session is not change to Committed |
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.
英文版md和标题提现一下saga
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
There is a follow-up issue that needs attention, success
is always true in AT transaction mode.
* '1103' of https://github.com/zw201913/seata: optimize: remove useless code (apache#5047) bugfix: fix startup failure of Server1.5.2 by using OpenJDK 11 (apache#4874) bugfix: insert value is all parsed as string in insert on duplicate (apache#5028) bugfix: global session is not change to Committed in saga mode (apache#5050)
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
fixes #5045
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews