-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
executor: fix bug when parsing the slow logs if the logNum is small (#20959) #21211
executor: fix bug when parsing the slow logs if the logNum is small (#20959) #21211
Conversation
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
/run-all-tests |
@ti-srebot you're already a collaborator in bot's repo. |
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
/merge |
Sorry @crazycs520, this branch cannot be merged without an approval of release maintainers |
/run-check_dev |
/run-all-tests |
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
/merge |
Sorry @crazycs520, this branch cannot be merged without an approval of release maintainers |
/merge |
Sorry @tiancaiamao, this branch cannot be merged without an approval of release maintainers |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@ti-srebot merge failed. |
/merge |
/run-all-tests |
cherry-pick #20959 to release-4.0
What problem does this PR solve?
Issue Number: close #20928
Problem Summary:
What is changed and how it works?
What's Changed:
fix bug when parsing the slow logs if the logNum is small
How it Work:
add judgment for checking whether a slow log has been parsed.
Check List
Tests
Release note