-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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](Nereids) support analyze database/table command #48031
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 |
run buildall |
TPC-H: Total hot run time: 31522 ms
|
TPC-DS: Total hot run time: 183253 ms
|
ClickBench: Total hot run time: 29.95 s
|
} | ||
Map<String, String> propertiesMap = new HashMap<>(); | ||
// default values | ||
propertiesMap.put(AnalyzeProperties.PROPERTY_SYNC, "false"); |
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.
why need set PROPERTY_SYNC to false as default value?
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.
why need set PROPERTY_SYNC to false as default value?
old planner need this as default value.
|
||
Map<String, String> propertiesMap = new HashMap<>(); | ||
// default values | ||
propertiesMap.put(AnalyzeProperties.PROPERTY_SYNC, "false"); |
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.
why need set PROPERTY_SYNC to false as default value?
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.
why need set PROPERTY_SYNC to false as default value?
old planner need this as default value.
fe/fe-core/src/main/java/org/apache/doris/statistics/AnalysisManager.java
Outdated
Show resolved
Hide resolved
run buildall |
TPC-H: Total hot run time: 31541 ms
|
TPC-DS: Total hot run time: 183515 ms
|
ClickBench: Total hot run time: 30.27 s
|
run buildall |
TPC-H: Total hot run time: 31718 ms
|
TPC-DS: Total hot run time: 190375 ms
|
ClickBench: Total hot run time: 31.13 s
|
run buildall |
TPC-H: Total hot run time: 31474 ms
|
TPC-DS: Total hot run time: 189722 ms
|
ClickBench: Total hot run time: 30.08 s
|
run cloud_p0 |
What problem does this PR solve?
Issue Number: close #42706 #42705
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)