This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new de2bc98fa2 [bugfix](compaction) remove useless check #17804
de2bc98fa2 is described below

commit de2bc98fa26d30136b9864275e184d28ae72e96b
Author: morningman <[email protected]>
AuthorDate: Thu Mar 16 17:48:24 2023 +0800

    [bugfix](compaction) remove useless check #17804
    
    cherry-pick #17804
---
 be/src/olap/cumulative_compaction.cpp | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/be/src/olap/cumulative_compaction.cpp 
b/be/src/olap/cumulative_compaction.cpp
index db9aaddf8a..54e4434fa0 100644
--- a/be/src/olap/cumulative_compaction.cpp
+++ b/be/src/olap/cumulative_compaction.cpp
@@ -117,7 +117,7 @@ Status CumulativeCompaction::pick_rowsets_to_compact() {
     }
 
     size_t compaction_score = 0;
-    int transient_size = 
_tablet->cumulative_compaction_policy()->pick_input_rowsets(
+    _tablet->cumulative_compaction_policy()->pick_input_rowsets(
             _tablet.get(), candidate_rowsets,
             config::max_cumulative_compaction_num_singleton_deltas,
             config::min_cumulative_compaction_num_singleton_deltas, 
&_input_rowsets,
@@ -151,9 +151,6 @@ Status CumulativeCompaction::pick_rowsets_to_compact() {
             if (cumu_interval > interval_threshold && base_interval > 
interval_threshold) {
                 // before increasing cumulative point, we should make sure all 
rowsets are non-overlapping.
                 // if at least one rowset is overlapping, we should compact 
them first.
-                CHECK(candidate_rowsets.size() == transient_size)
-                        << "tablet: " << _tablet->full_name() << ", " << 
candidate_rowsets.size()
-                        << " vs. " << transient_size;
                 for (auto& rs : candidate_rowsets) {
                     if (rs->rowset_meta()->is_segments_overlapping()) {
                         _input_rowsets = candidate_rowsets;


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to