10

be/src/cloud/cloud_cumulative_compaction.cpp

@@ -92,7 +92,7 @@ Status CloudCumulativeCompaction::prepare_compact() {
             // plus 1 to skip the delete version.
             // NOTICE: after that, the cumulative point may be larger than max version of this tablet, but it doesn't matter.
             update_cumulative_point();
-            if (config::enable_sleep_between_delete_cumu_compaction) {
+            if (!config::enable_sleep_between_delete_cumu_compaction) {
                 st = Status::Error<CUMULATIVE_NO_SUITABLE_VERSION>(
                         "_last_delete_version.first not equal to -1");
             }

regression-test/data/compaction/test_cumu_compaction_with_delete.out

@@ -1,3 +1,5 @@
 -- This file is automatically generated. You should know what you did if you want to edit this
--- !select --
+-- !select1 --
+
+-- !select2 --
 

regression-test/suites/compaction/test_cumu_compaction_with_delete.groovy

@@ -60,9 +60,9 @@ suite("test_cumu_compaction_with_delete") {
         }
         time_diff = System.currentTimeMillis() - now
         logger.info("time_diff:" + time_diff)
-        assertTrue(time_diff<250*1000)
+        assertTrue(time_diff<200*1000)
 
-        qt_select """select * from ${tableName} order by user_id, value"""
+        qt_select1 """select * from ${tableName} order by user_id, value"""
     } catch (Exception e){
         logger.info(e.getMessage())
         assertFalse(true)
@@ -111,7 +111,7 @@ suite("test_cumu_compaction_with_delete") {
         logger.info("time_diff:" + time_diff)
         assertTrue(time_diff>=250*1000)
 
-        qt_select """select * from ${tableName} order by user_id, value"""
+        qt_select2 """select * from ${tableName} order by user_id, value"""
     } catch (Exception e){
         logger.info(e.getMessage())
         assertFalse(true)