-
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
session/statistics: discard feedbacks from `delete ... (#17452) #17843
session/statistics: discard feedbacks from `delete ... (#17452) #17843
Conversation
Signed-off-by: sre-bot <sre-bot@pingcap.com>
/run-all-tests |
/merge |
Sorry @you06, you don't have permission to trigger auto merge event on this branch. |
/merge |
/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
/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 |
/run-all-tests |
/run-all-tests |
/run-all-tests |
@sre-bot merge failed. |
/run-all-tests |
@sre-bot merge failed. |
/run-sqllogic-test-2 |
delete
/ update
(#17452)
cherry-pick #17452 to release-4.0
What problem does this PR solve?
Issue Number: close #17438
Problem Summary:
Feedbacks from
update
,delete
may be invalid.What is changed and how it works?
What's Changed:
Discard those feedbacks.
How it Works:
Related changes
Check List
Tests
Side effects
N/A
Release note
delete
/update
statements.