Skip to content

branch-2.1: [fix](Nereids) handle continuous filter or project in plan #40176 - #54870

Merged
yiguolei merged 3 commits into
apache:branch-2.1from
yujun777:pick_40176_to_upstream_branch-2.1
Aug 22, 2025
Merged

branch-2.1: [fix](Nereids) handle continuous filter or project in plan #40176#54870
yiguolei merged 3 commits into
apache:branch-2.1from
yujun777:pick_40176_to_upstream_branch-2.1

Conversation

@yujun777

@yujun777yujun777 commented Aug 15, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

cherry pick from #40176, #54872

@morrySnow@yujun777
[fix](Nereids) handle continuous filter or project in plan (apache#40176
fcef247
)
if we meet continuous project or filter in translator, we try to
generate SelectNode as far as possible to avoid generate invalid plan
for example
```
Filter(conjuncts 1)
+-- Limit (limit 10)
+-- Filter(conjuncts 2)
+-- Aggregate
```
will be translated to
```
SELECT_NODE (conjuncts 1)
+-- AGGREGATE_NODE (conjuncts 2) (limit 10)
```
@yujun777

yujun777 commented Aug 15, 2025

Copy link
Copy Markdown
ContributorAuthor

run buildall

@hello-stephen

hello-stephen commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@yujun777

yujun777 commented Aug 15, 2025

Copy link
Copy Markdown
ContributorAuthor

run buildall

@yujun777

yujun777 commented Aug 15, 2025

Copy link
Copy Markdown
ContributorAuthor

run p0

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Aug 20, 2025
@github-actions

github-actionsBot commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions

github-actionsBot commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@yiguolei
yiguolei merged commit a632e7a into apache:branch-2.1Aug 22, 2025
20 of 21 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yiguoleiyiguoleiyiguolei approved these changes
@morrySnowmorrySnowmorrySnow approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yujun777@hello-stephen@yiguolei@morrySnow