-
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
[opt](mtmv) Support null aware left anti join rewrite by materialized view #45815
base: master
Are you sure you want to change the base?
[opt](mtmv) Support null aware left anti join rewrite by materialized view #45815
Conversation
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
run buildall |
TPC-H: Total hot run time: 40400 ms
|
TPC-DS: Total hot run time: 191226 ms
|
ClickBench: Total hot run time: 32.04 s
|
LOG.info("computeOlapScan optStats isPresent {}, tableRowCount is {}", | ||
optStats.isPresent(), tableRowCount); | ||
if (optStats.isPresent()) { | ||
double selectedPartitionsRowCount = getSelectedPartitionRowCount(olapScan, tableRowCount); | ||
LOG.info("computeOlapScan optStats is {}, selectedPartitionsRowCount is {}", optStats.get(), | ||
selectedPartitionsRowCount); |
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.
remove these logs
run buildall |
TPC-H: Total hot run time: 32246 ms
|
TPC-DS: Total hot run time: 189740 ms
|
ClickBench: Total hot run time: 31.32 s
|
c0c2c81
to
78f4262
Compare
run buildall |
TPC-H: Total hot run time: 32856 ms
|
TPC-DS: Total hot run time: 194106 ms
|
ClickBench: Total hot run time: 31.58 s
|
need more test cases |
run buildall |
TPC-H: Total hot run time: 32573 ms
|
TPC-DS: Total hot run time: 187981 ms
|
ClickBench: Total hot run time: 30.5 s
|
run buildall |
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
PR approved by anyone and no changes requested. |
TPC-H: Total hot run time: 33049 ms
|
TPC-DS: Total hot run time: 187609 ms
|
ClickBench: Total hot run time: 30.99 s
|
What problem does this PR solve?
Support null aware left anti join rewrite by materialized view
such as mv def is as following, it's join type is NULL_AWARE_LEFT_ANTI_JOIN
if query is as following, should be rewritten successlly, this pr should this
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
Support null aware left anti join rewrite by materialized view
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)