-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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: optimize collectRowLocks
method.
#3909
optimize: optimize collectRowLocks
method.
#3909
Conversation
可以把源码中所有类似的地方都找到一并修改吗? |
Sorry, this is all I can see so far. I will continue to submit if I see any more. |
ok, we'll wait for you to add it to this pr |
I think we should merge the branches first and make sure that each branch only does one thing. No similar code has been found anywhere else. |
I think it's the same thing |
Codecov Report
@@ Coverage Diff @@
## develop #3909 +/- ##
==========================================
Coverage 49.39% 49.39%
Complexity 3726 3726
==========================================
Files 692 692
Lines 23435 23436 +1
Branches 2911 2911
==========================================
+ Hits 11576 11577 +1
Misses 10693 10693
Partials 1166 1166
|
collectRowLocks
method.collectRowLocks
method.
Ⅰ. Describe what this PR did
Ⅱ. 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