yx-keith opened a new pull request, #64207:
URL: https://github.com/apache/doris/pull/64207
…s disabled (#59593)
When a partitioned mv misses some queried partitions, union compensation
with the base table is required to produce a complete result. The
partition-compensation branch was gated by
enable_materialized_view_union_rewrite, so when that flag was false the whole
branch was skipped and the partial mv-only plan was used as the rewrite result,
silently dropping the rows of the uncovered partitions.
Compute the invalid partitions regardless of the flag, and bail out (the mv
can not be used for this query) when the mv actually needs union compensation
but union rewrite is disabled - same handling as when union compensation can
not be done because of the query structInfo. When the mv covers all queried
partitions (no union needed) the mv is still used as before, so the flag only
affects mvs that genuinely require union.
Add a regression test that, with the flag off and an mv missing a base-table
partition, asserts the mv is not chosen and the query still returns the
complete result.
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
