-
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: set the correct stmtCtx for explain statement #11186
Conversation
WAIT FOR #11102 |
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.
please fix CI..
@XuHuaiyu Friendly ping, any update? |
/run-all-tests |
Ready to be reviewed. @zz-jason @qw4990 @SunRunAway @wshwsh12 |
Codecov Report
@@ Coverage Diff @@
## master #11186 +/- ##
=========================================
Coverage 81.281% 81.281%
=========================================
Files 438 438
Lines 94685 94685
=========================================
Hits 76961 76961
Misses 12226 12226
Partials 5498 5498 |
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 |
@XuHuaiyu merge failed. |
/run-unit-test |
/merge |
/run-all-tests |
cherry pick to release-2.1 failed |
cherry pick to release-3.0 failed |
What problem does this PR solve?
Fix #11124
What is changed and how it works?
Check the statement type of a explain statement, and set the correct stmtCtx for it.
Check List
Tests
TO BE ADDED.
Code changes
Side effects
N/A
Related changes