-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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: the problem that the number of mirrors before and after may be inconsistent #4748
base: develop
Are you sure you want to change the base?
Conversation
|
Codecov Report
@@ Coverage Diff @@
## develop #4748 +/- ##
=============================================
+ Coverage 49.29% 49.31% +0.02%
- Complexity 4126 4144 +18
=============================================
Files 736 737 +1
Lines 26130 26199 +69
Branches 3226 3241 +15
=============================================
+ Hits 12880 12921 +41
- Misses 11870 11894 +24
- Partials 1380 1384 +4
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #4748 +/- ##
==========================================
Coverage ? 48.36%
Complexity ? 4162
==========================================
Files ? 795
Lines ? 28155
Branches ? 3442
==========================================
Hits ? 13617
Misses ? 13109
Partials ? 1429
|
Ⅰ. Describe what this PR did
1.update count大于 before size时,抛出异常
2.当update的where条件中没有主键时,后镜像采用前镜像方式制作,且进行对比前后镜像查询的行数结果
Ⅱ. Does this pull request fix one issue?
fixes #4727
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews