-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[fix](nereids)build agg for random distributed agg table in bindRelation phase #40181
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
TPC-H: Total hot run time: 37640 ms
|
TPC-DS: Total hot run time: 192856 ms
|
ClickBench: Total hot run time: 32.16 s
|
6db47b5
to
1fa84fb
Compare
run buildall |
TPC-H: Total hot run time: 37923 ms
|
TPC-DS: Total hot run time: 193628 ms
|
ClickBench: Total hot run time: 32.03 s
|
1fa84fb
to
506c262
Compare
run buildall |
TPC-H: Total hot run time: 37820 ms
|
TPC-DS: Total hot run time: 191810 ms
|
ClickBench: Total hot run time: 31.57 s
|
fe/fe-core/src/main/java/org/apache/doris/nereids/jobs/executor/Analyzer.java
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindRelation.java
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindRelation.java
Show resolved
Hide resolved
run buildall |
TPC-H: Total hot run time: 38306 ms
|
TPC-DS: Total hot run time: 188408 ms
|
ClickBench: Total hot run time: 31.73 s
|
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
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.
This change maybe conflict to CheckPolicy because it only support filter-relation or relation, please add row policy / data masking test case
4330453
to
177f1d0
Compare
ClickBench: Total hot run time: 31.28 s
|
177f1d0
to
0394a85
Compare
run buildall |
TPC-H: Total hot run time: 37788 ms
|
TPC-DS: Total hot run time: 192745 ms
|
ClickBench: Total hot run time: 32.09 s
|
0394a85
to
4614578
Compare
run buildall |
TPC-H: Total hot run time: 38691 ms
|
TPC-H: Total hot run time: 38111 ms
|
TPC-DS: Total hot run time: 197162 ms
|
ClickBench: Total hot run time: 31.77 s
|
PR approved by at least one committer and no changes requested. |
it's better to build agg for random distributed agg table in bindRelation phase instead of in BuildAggForRandomDistributedTable RULE
Proposed changes
Issue Number: close #xxx