[PR] [fix](mv) Fix use sync mv wrongly when use rbo materialized view rewrite rule [doris]

[via GitHub]

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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org