[GitHub] [incubator-doris] chaoyli commented on a change in pull request #2473: Optimize compaction strategy of tablet on BE

[GitBox <gi...@apache.org>]

chaoyli commented on a change in pull request #2473: Optimize compaction strategy of tablet on BE
URL: https://github.com/apache/incubator-doris/pull/2473#discussion_r358176761
 
 

 ##########
 File path: be/src/olap/cumulative_compaction.cpp
 ##########
 @@ -76,17 +76,22 @@ OLAPStatus CumulativeCompaction::pick_rowsets_to_compact() {
 
     std::vector<RowsetSharedPtr> transient_rowsets;
     size_t num_overlapping_segments = 0;
+    // the last delete version we meet when traversing candidate_rowsets
+    Version last_delete_version { -1, -1 };
+
+    // traverse rowsets from begin to penultimate rowset.
+    // Becuse VersionHash will calculated from chosen rowsets.
 
 Review comment:
   Because

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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