Repository navigation
[fix](virtual column) should not push down WhenClause in case when - #54998
Merged
Merged
Conversation
Contributor
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
Contributor
Author
|
run buildall |
TPC-H: Total hot run time: 33770 ms |
TPC-DS: Total hot run time: 185087 ms |
ClickBench: Total hot run time: 32.05 s |
morrySnow
force-pushed
the
fix_push_when_to_virtual
branch
from
August 21, 2025 03:35
053f0c5 to
ebeac49
Compare
Contributor
Author
|
run buildall |
TPC-H: Total hot run time: 33813 ms |
TPC-DS: Total hot run time: 185615 ms |
ClickBench: Total hot run time: 32.78 s |
Contributor
FE UT Coverage ReportIncrement line coverage |
morrySnow
force-pushed
the
fix_push_when_to_virtual
branch
from
August 27, 2025 03:34
ebeac49 to
32303dd
Compare
Contributor
Author
|
run buildall |
TPC-H: Total hot run time: 33966 ms |
TPC-DS: Total hot run time: 186879 ms |
ClickBench: Total hot run time: 32.61 s |
Contributor
FE UT Coverage ReportIncrement line coverage |
RoanHeNaN
approved these changes
Aug 28, 2025
RoanHeNaN
left a comment
Contributor
There was a problem hiding this comment.
LGTM, fix the pattern like select func_a(x), func_b(func_a(x)), func_c(func_b(func_a(x))).
Contributor
|
PR approved by anyone and no changes requested. |
starocean999
approved these changes
Aug 28, 2025
morrySnow
marked this pull request as ready for review
August 28, 2025 07:46
Contributor
|
PR approved by at least one committer and no changes requested. |
RoanHeNaN
pushed a commit
to RoanHeNaN/doris
that referenced
this pull request
Aug 29, 2025
…pache#54998) ### What problem does this PR solve? Related PR: apache#52701 Problem Summary: 1. do not push down WhenClause in case when 2. not generate virtual column which only used once: fix the pattern like `select func_a(x), func_b(func_a(x)), func_c(func_b(func_a(x)))`.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What problem does this PR solve?
Related PR: #52701
Problem Summary:
select func_a(x), func_b(func_a(x)), func_c(func_b(func_a(x))).Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)