-
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 showStoragePolicyCommand in nereids #44393
base: master
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
run buildall |
TPC-H: Total hot run time: 40179 ms
|
TPC-DS: Total hot run time: 197673 ms
|
ClickBench: Total hot run time: 33.11 s
|
run buildall |
TPC-H: Total hot run time: 39743 ms
|
TPC-DS: Total hot run time: 195937 ms
|
ClickBench: Total hot run time: 32.01 s
|
run p0 |
run buildall |
TPC-H: Total hot run time: 40140 ms
|
TPC-DS: Total hot run time: 196783 ms
|
ClickBench: Total hot run time: 32.86 s
|
run buildall |
TPC-H: Total hot run time: 39951 ms
|
TPC-DS: Total hot run time: 197703 ms
|
ClickBench: Total hot run time: 33.13 s
|
run buildall |
run P0 |
pls check cases failed in p0, it seems related to changed |
run buildall |
TPC-H: Total hot run time: 40149 ms
|
TPC-DS: Total hot run time: 198082 ms
|
ClickBench: Total hot run time: 32.89 s
|
0d1d9db
to
bd82857
Compare
run buildall |
TPC-H: Total hot run time: 39550 ms
|
TPC-DS: Total hot run time: 196085 ms
|
ClickBench: Total hot run time: 32.87 s
|
@@ -5031,6 +5033,11 @@ public LogicalPlan visitShowDelete(ShowDeleteContext ctx) { | |||
return new ShowDeleteCommand(dbName); | |||
} | |||
|
|||
@Override | |||
public LogicalPlan visitShowStoragePolicy(ShowStoragePolicyContext ctx) { | |||
return new ShowStoragePolicyCommand(ctx.identifierOrText().getText()); |
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.
policy name is optional, please check if policy exists
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 - please review
bd82857
to
9f787c9
Compare
run buildall |
TPC-H: Total hot run time: 39791 ms
|
TPC-DS: Total hot run time: 195757 ms
|
ClickBench: Total hot run time: 32.08 s
|
What problem does this PR solve?
Issue Number: close #42721
Problem Summary:
[Enhancement] (nereids)implement showStoragePolicyCommand in nereids
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)