adonis0147 opened a new pull request, #16006:
URL: https://github.com/apache/doris/pull/16006

   # Proposed changes
   
   1. Remove some redundant code.
   2. Fix the issue with the state of MTMV task.
   3. Fix the case - test_create_mtmv.
   
   ## Problem summary
   
   1. We used a retry policy to re-run the failed MTMV tasks, but we set the 
state to `FAILURE` during re-running the tasks. We should do this after all the 
retry runs fail.
   2. There are some redundant code can be removed.
   3. In the case test_create_mtmv, we created many background tasks to refresh 
the data. Some task may fail due to the concurrency and cause the test fail. 
Actually, we only need single one task to verify the functionality.
   
   ## Checklist(Required)
   
   1. Does it affect the original behavior: 
       - [ ] Yes
       - [ ] No
       - [x] I don't know
   4. Has unit tests been added:
       - [x] Yes
       - [ ] No
       - [ ] No Need
   5. Has document been added or modified:
       - [ ] Yes
       - [ ] No
       - [x] No Need
   6. Does it need to update dependencies:
       - [ ] Yes
       - [x] No
   7. Are there any changes that cannot be rolled back:
       - [ ] Yes (If Yes, please explain WHY)
       - [x] No
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at 
[[email protected]](mailto:[email protected]) by explaining why you 
chose the solution you did and what alternatives you considered, etc...
   
   


-- 
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]

Reply via email to