-
Notifications
You must be signed in to change notification settings - Fork 1.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
[Enhancement] add explain costs in query detail (backport #51439) #51481
Conversation
Cherry-pick of 92dfb5d has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
@mergify[bot]: Backport conflict, please reslove the conflict and resubmit the pr |
7fb3aab
to
b32218a
Compare
Signed-off-by: Murphy <mofei@starrocks.com> (cherry picked from commit 92dfb5d) Signed-off-by: Murphy <mofei@starrocks.com>
b32218a
to
0ec2e4c
Compare
Quality Gate passedIssues Measures |
Why I'm doing:
EXPLAIN COSTS
is much more useful to debug historical query,EXPLAIN
usually is not enough.What I'm doing:
EXPLAIN COSTS
in query detail instead ofEXPLAIN
query_detail_explain_level
to control the behaviorFixes #issue
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check:
This is an automatic backport of pull request #51439 done by [Mergify](https://mergify.com). ## Why I'm doing:
EXPLAIN COSTS
is much more useful to debug historical query,EXPLAIN
usually is not enough.What I'm doing:
EXPLAIN COSTS
in query detail instead ofEXPLAIN
query_detail_explain_level
to control the behaviorFixes #issue
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist: