Skip to content

[fix](mtmv) Eliminate redundant filters in nested MV candidate defini… - #69381

Merged
yiguolei merged 4 commits into
apache:branch-4.2from
foxtail463:fix/nested-mv-relation-implied-predicates-4.2
Oct 29, 2026
Merged

[fix](mtmv) Eliminate redundant filters in nested MV candidate defini…#69381
yiguolei merged 4 commits into
apache:branch-4.2from
foxtail463:fix/nested-mv-relation-implied-predicates-4.2

Conversation

@foxtail463

@foxtail463 foxtail463 commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Cherry-picked from #63639

@foxtail463
foxtail463 requested a review from yiguolei as a code owner October 21, 2026 07:50
@hello-stephen

hello-stephen commented Oct 21, 2026

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?

@foxtail463

foxtail463 commented Oct 21, 2026

Copy link
Copy Markdown
Contributor Author

run buildall

@foxtail463

foxtail463 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

run buildall

yiguolei
yiguolei previously approved these changes Oct 22, 2026
View reviewed changes
@foxtail463

foxtail463 commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

run buildall

@hello-stephen

hello-stephen commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

FE UT Coverage Report

Increment line coverage 67.65% (92/136) 🎉
Increment coverage report
Complete coverage report

@hello-stephen

hello-stephen commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

FE Regression Coverage Report

Increment line coverage 84.56% (115/136) 🎉
Increment coverage report
Complete coverage report

@foxtail463

foxtail463 commented Oct 23, 2026

Copy link
Copy Markdown
Contributor Author

run feut

1 similar comment
@foxtail463

foxtail463 commented Oct 23, 2026

Copy link
Copy Markdown
Contributor Author

run feut

@foxtail463

foxtail463 commented Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

run buildall

@hello-stephen

hello-stephen commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

FE UT Coverage Report

Increment line coverage 87.94% (124/141) 🎉
Increment coverage report
Complete coverage report

@hello-stephen

hello-stephen commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

FE Regression Coverage Report

Increment line coverage 84.40% (119/141) 🎉
Increment coverage report
Complete coverage report

@yiguolei
yiguolei merged commit ea2df20 into apache:branch-4.2 Oct 29, 2026
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yiguolei yiguolei yiguolei approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@foxtail463 @hello-stephen @yiguolei