-
Notifications
You must be signed in to change notification settings - Fork 3.6k
[fix](nereids) adjust distribute expr lists after project common sub expression for aggregation #57258
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
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 |
fe/fe-core/src/main/java/org/apache/doris/nereids/properties/ChildOutputPropertyDeriver.java
Show resolved
Hide resolved
|
run buildall |
1 similar comment
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
ClickBench: Total hot run time: 28.78 s |
FE Regression Coverage ReportIncrement line coverage |
48fad2a to
3e53f1d
Compare
|
run buildall |
ClickBench: Total hot run time: 28.25 s |
|
run vault_p0 |
FE Regression Coverage ReportIncrement line coverage |
What problem does this PR solve?
after extract common sub expression, some slots are not output slots any more. But if these slots are used in distribute expression list, wich is used by local shuffle, BE will report slot not found exception.
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)