hust-hhb opened a new pull request, #22599:
URL: https://github.com/apache/doris/pull/22599

   ## Proposed changes
   when a version publish late, cmululative compaction won't pick corresponding 
rowset to do compaction, because the version won't be in tablet 
_rs_version_map. After this round cmululative compaction, the cmululative point 
will update and the late publish version will never have chance to do 
cmululative compaction unless restar be to recalculate cmululative point. 
However now base compaction will check rowset overlapping first, and it will 
fail in this situation. After checking the code, i think check overlapping can 
remove in base compaction, and check version continuity is enough. The diffrent 
between base compaction and cmululative compaction is that base compaction will 
filter deleted row, and block reader they are used is same, so no need to worry 
about data aggregation in agg type.
   Issue Number: close #xxx
   
   <!--Describe your changes.-->
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at 
[d...@doris.apache.org](mailto:d...@doris.apache.org) 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: 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

Reply via email to