-
Notifications
You must be signed in to change notification settings - Fork 315
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
feat: support explain
syntax
#546
Conversation
explain
syntaxexplain
syntax
@v0y4g3r Hi Lei, Could you review this PR, If you have time. |
Glad to help. We will finish the review asap. |
Codecov Report
@@ Coverage Diff @@
## develop #546 +/- ##
===========================================
- Coverage 86.41% 86.30% -0.11%
===========================================
Files 400 402 +2
Lines 50650 50886 +236
===========================================
+ Hits 43770 43918 +148
- Misses 6880 6968 +88
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@francis-du Could you please fix the failing clippy check? You can test before push by running following command in your own machine: cargo clippy --workspace --all-targets -- -D warnings -D clippy::print_stdout -D clippy::print_stderr |
cargo clippy validation passed locally |
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
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Support explain syntax like this:
Checklist
explain
to sql parserexplain
test to sql parserexplain
in DataNodeexplain
in FrontendRefer to a related PR or issue link (optional)
Closes #531