-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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] (nereids)implement helpCommand in nereids #44819
Conversation
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
run buildall |
TPC-H: Total hot run time: 39922 ms
|
TPC-DS: Total hot run time: 197267 ms
|
ClickBench: Total hot run time: 33.23 s
|
pls resolved conflicts |
run buildall |
TPC-H: Total hot run time: 40475 ms
|
TPC-DS: Total hot run time: 197794 ms
|
ClickBench: Total hot run time: 32.48 s
|
run buildall |
TPC-H: Total hot run time: 40356 ms
|
TPC-DS: Total hot run time: 196423 ms
|
ClickBench: Total hot run time: 33.01 s
|
run buildall |
1 similar comment
run buildall |
TPC-H: Total hot run time: 39494 ms
|
TPC-DS: Total hot run time: 196581 ms
|
ClickBench: Total hot run time: 31.55 s
|
return visitor.visitHelpCommand(this, context); | ||
} | ||
|
||
public ShowResultSetMetaData getMetaData() { |
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.
public ShowResultSetMetaData getMetaData() { |
better to remove all 3 getXXXMetaData() methods, and use the member variable directly in doRun method
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.
done - pls review
461df0d
to
9b978cf
Compare
run buildall |
9b978cf
to
28cee7f
Compare
run buildall |
TPC-H: Total hot run time: 39945 ms
|
TPC-DS: Total hot run time: 196137 ms
|
ClickBench: Total hot run time: 32.52 s
|
run buildall |
TPC-H: Total hot run time: 40075 ms
|
TPC-DS: Total hot run time: 196748 ms
|
ClickBench: Total hot run time: 33.02 s
|
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
What problem does this PR solve?
Issue Number: close #42824
Related PR: #xxx
Problem Summary:
[Enhancement] (nereids)implement helpCommand in nereids
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)