-
Notifications
You must be signed in to change notification settings - Fork 5.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
planner: encode insert/delete/update executor plan ... (#19176) #19269
planner: encode insert/delete/update executor plan ... (#19176) #19269
Conversation
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
/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.
pls resolve the conflicts
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.
@crazycs520 please resolve conflicts and fix CI.
…lease-4.0-67214e763f23
/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
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
/run-all-tests |
/merge |
Your auto merge job has been accepted, waiting for:
|
/run-all-tests |
@ti-srebot merge failed. |
/run-unit-test |
/run-unit-test |
/run-all-tests |
/run-unit-test |
3 similar comments
/run-unit-test |
/run-unit-test |
/run-unit-test |
/run-all-tests |
/run-all-tests |
cherry-pick #19176 to release-4.0
What problem does this PR solve?
Before this PR, TiDB doesn't record the
insert/update/delete
executor plan information in thePlan
field of slow-log file.here is some example of
Plan
field in slow-log file:Attention
Currently, the
insert/update/delete
executor runtime information doesn't contain the transaction commit durationWhat is changed and how it works?
Related changes
Check List
Tests
Side effects
Release note