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: intercept insert on duplicate update statement which has no … #5075
optimize: intercept insert on duplicate update statement which has no … #5075
Conversation
…primary key and unique index
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
Codecov Report
@@ Coverage Diff @@
## develop #5075 +/- ##
=============================================
- Coverage 49.46% 49.42% -0.05%
+ Complexity 4128 4126 -2
=============================================
Files 734 734
Lines 26058 26061 +3
Branches 3221 3220 -1
=============================================
- Hits 12890 12880 -10
- Misses 11791 11805 +14
+ Partials 1377 1376 -1
|
不同数据库,对
例如:
|
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
…o 1114_for_5073 * '1114_for_5073' of https://github.com/zw201913/seata: test: test support for arm64 (apache#5069) test: add AT transaction mode tests to support database data types (Oracle) (apache#4411) bugfix: fix could not intercept the sql witch has no primary and unique (apache#5078) optimize: intercept insert on duplicate update statement which has no primary key and unique index (apache#5075) optimize: support oracle date types (apache#4542)
…primary key and unique index
Ⅰ. Describe what this PR did
针对主键自增且没有唯一索引值的insert on duplicate语句,由于主键自增值依赖驱动版本情况,需要再测试(#5065),1.6版本先进行拦截。后面的版本再考虑合入(https://github.com/seata/seata/pull/5044)
Ⅱ. Does this pull request fix one issue?
fixes #5007
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews