seawinde opened a new pull request, #47650:
URL: https://github.com/apache/doris/pull/47650
### What problem does this PR solve?
if mv def sql is as following:
```sql
create matrialized view test_mv as
select k1, k3, sum(k2), count(k4) from ${tblName} group by k1, k3;
```
if query are as following, should not use test_mv above
```sql
select count(k1) from agg_use_key_direct;
select sum(k1) from agg_use_key_direct
select avg(k1) from agg_use_key_direct
```
the wrong case would happen when in branch-2.1 or `set
enable_sync_mv_cost_based_rewrite = false;` on branch-3.0 or master, the pr fix
this
if query are as following with distinct or is `max` or `min`, can use
test_mv above
```sql
select count(distinct k1) from agg_use_key_direct;
select sum(distinct k1) from agg_use_key_direct
select avg(distinct k1) from agg_use_key_direct
select max(k1) from agg_use_key_direct
select min(k1) from agg_use_key_direct
```
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
Fix use sync mv wrongly when use rbo materialized view rewrite rule
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [x] 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]
