Uh oh!
There was an error while loading. Please reload this page.
[opt](nereids) sum literal rewrite rule support non slot expression - #55543
Conversation
yujun777
commented
Sep 1, 2025
run buildall |
doris-robot
commented
Sep 1, 2025
TPC-H: Total hot run time: 34033 ms |
doris-robot
commented
Sep 1, 2025
TPC-DS: Total hot run time: 186868 ms |
hello-stephen
commented
Sep 1, 2025
FE UT Coverage ReportIncrement line coverage |
49d739c to
6dbaef8Compareyujun777
commented
Sep 1, 2025
run buildall |
doris-robot
commented
Sep 1, 2025
TPC-H: Total hot run time: 33905 ms |
doris-robot
commented
Sep 1, 2025
TPC-DS: Total hot run time: 187659 ms |
hello-stephen
commented
Sep 1, 2025
FE UT Coverage ReportIncrement line coverage |
hello-stephen
commented
Sep 1, 2025
FE Regression Coverage ReportIncrement line coverage |
yujun777
commented
Sep 2, 2025
run p0 |
yujun777
commented
Sep 2, 2025
run performance |
doris-robot
commented
Sep 2, 2025
TPC-H: Total hot run time: 33888 ms |
doris-robot
commented
Sep 2, 2025
TPC-DS: Total hot run time: 187821 ms |
hello-stephen
commented
Sep 2, 2025
FE Regression Coverage ReportIncrement line coverage |
yujun777
commented
Sep 4, 2025
run buildall |
doris-robot
commented
Sep 4, 2025
TPC-H: Total hot run time: 34268 ms |
doris-robot
commented
Sep 4, 2025
TPC-DS: Total hot run time: 187308 ms |
doris-robot
commented
Sep 4, 2025
ClickBench: Total hot run time: 30.72 s |
hello-stephen
commented
Sep 4, 2025
FE UT Coverage ReportIncrement line coverage |
hello-stephen
commented
Sep 4, 2025
FE Regression Coverage ReportIncrement line coverage |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Uh oh!
There was an error while loading. Please reload this page.
…pache#55543) rule SumLiteralRewrite will rewrite `sum(expr +/- literal) ` ==> `sum(expr) +/- literal * count(expr)` but require `expr` is a slot, this PR remove this restriction, allow expr to be any expression. relate PR: - introduce rule SumLiteralRewrite: apache#32244 - add cast for `+,-,*, /`, then `sum(a + 10)` will become `sum(cast(a as bigint) + 10)`: apache#52837
What problem does this PR solve?
rule SumLiteralRewrite will rewrite
sum(expr +/- literal)==>sum(expr) +/- literal * count(expr)but require
expris a slot, this PR remove this restriction, allow expr to be any expression.relate PR:
+,-,*, /, thensum(a + 10)will becomesum(cast(a as bigint) + 10): [refactor](divide) Reduce template args for divide operation #52837Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)