-
Notifications
You must be signed in to change notification settings - Fork 287
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
[seata-java] bugfix: mysql insert on update duplicate sensitive case not matched #700
Comments
plz assign me, it is leehao |
这个seata-java中的问题,在seata-go中不会出现。 在构造Column后,会使用小写。 incubator-seata-go/pkg/datasource/sql/exec/at/insert_on_update_executor.go Lines 395 to 397 in ad092d5
我认为可以关闭这个issue。 |
👍🏻,反过来,假设SQL是大写,这里自动使用column的小写,是否会有问题? |
…che#700) Signed-off-by: LeeHao <[email protected]>
抱歉这么久才来回复您,确实会有问题,在新的pr中已经修复,烦请review。 |
seata-java: apache/incubator-seata#4912
Ⅰ. Describe what this PR did
当数据库column为大写,而sql为小写时,读取parameter时匹配不上,造成了前镜像的sql为select * from table导致了全表扫描,后镜像也进行了一次全表扫描的bug
Ⅱ. Does this pull request fix one issue?
fixes #4909
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
The text was updated successfully, but these errors were encountered: