-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[BugFix] Fix pushdown subfield with unstrict expression in project error (backport #51100) #51135
Conversation
Cherry-pick of 2a85b7c has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
@mergify[bot]: Backport conflict, please reslove the conflict and resubmit the pr |
48d91ba
to
ea5ddbf
Compare
…ror (#51100) Signed-off-by: Seaven <[email protected]> (cherry picked from commit 2a85b7c)
ea5ddbf
to
d6bf276
Compare
Quality Gate passedIssues Measures |
Why I'm doing:
For unstrict expression with semi-type, e.g:
coalesce(array...)/ifnull(array...)
, will push down the whole expression cross outer join, it's the wrong way, will take the result errorwe should only push down
array
expression, notcoalesce(array...)
,coalesce
shoule stay in placeWhat I'm doing:
Fixes #issue
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check:
This is an automatic backport of pull request #51100 done by [Mergify](https://mergify.com). ## Why I'm doing:
For unstrict expression with semi-type, e.g:
coalesce(array...)/ifnull(array...)
, will push down the whole expression cross outer join, it's the wrong way, will take the result errorwe should only push down
array
expression, notcoalesce(array...)
,coalesce
shoule stay in placeWhat I'm doing:
Fixes #issue
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist: