Skip to content

branch-4.2: [test](mv) Add MV residual compensation coverage #64051 - #68389

Open
yujun777 wants to merge 1 commit into
apache:branch-4.2from
yujun777:pick-64051-mv-residual-compensation-4.2
Open

yujun777 wants to merge 1 commit into
apache:branch-4.2from
yujun777:pick-64051-mv-residual-compensation-4.2

Conversation

@yujun777

Copy link
Copy Markdown
Contributor

cherry-pick: #64051

Related PR: apache#61345

This PR adds focused FE unit test coverage for MV residual predicate
compensation.

The new cases cover:
- Comparable literal types beyond bigint, including date, timestamp,
decimal, and string literals.
- Boundary operator implication, such as `<` implying `<=`, `>` implying
`>=`, and `=` implying range predicates.
- Negative reverse cases to ensure broader predicates are not
incorrectly treated as covered.
- Nested DNF predicates where every expanded query branch must be
covered by a view branch.
- A mixed DNF negative case where adding one uncovered branch makes the
residual compensation unsafe.

No optimizer behavior is changed. This is test-only coverage for the
existing MV predicate compensation logic.

Co-authored-by: yangtao555 <yangtao555@jd.com>
@yujun777
yujun777 requested a review from yiguolei as a code owner September 22, 2026 09:42
@hello-stephen

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

Copy link
Copy Markdown
Contributor Author

run buildall

@yujun777

Copy link
Copy Markdown
Contributor Author

run p0

@yujun777

Copy link
Copy Markdown
Contributor Author

run cloud_p0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants