branch-3.0: [opt](test) Make the case stable #44910 (#45007)

Cherry-picked from #44910 Co-authored-by: Sun Chenyang <[email protected]>
apache · Dec 11, 2024 · 635aa6c · 635aa6c
1 parent 112bbb8
commit 635aa6c
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/...on-test/suites/inverted_index_p0/array_contains/test_index_compaction_dup_keys_arr.groovy b/...on-test/suites/inverted_index_p0/array_contains/test_index_compaction_dup_keys_arr.groovy
@@ -17,7 +17,7 @@
 
 import org.codehaus.groovy.runtime.IOGroovyMethods
 
-suite("test_index_compaction_dup_keys_array", "array_contains_inverted_index") {
+suite("test_index_compaction_dup_keys_array", "array_contains_inverted_index, nonConcurrent") {
     // here some variable to control inverted index query
     sql """ set enable_profile=true"""
     sql """ set enable_pipeline_x_engine=true;"""

diff --git a/...ession-test/suites/inverted_index_p0/array_contains/test_index_compaction_null_arr.groovy b/...ession-test/suites/inverted_index_p0/array_contains/test_index_compaction_null_arr.groovy
@@ -17,7 +17,7 @@
 
 import org.codehaus.groovy.runtime.IOGroovyMethods
 
-suite("test_index_compaction_null_arr", "array_contains_inverted_index") {
+suite("test_index_compaction_null_arr", "array_contains_inverted_index, nonConcurrent") {
     // here some variable to control inverted index query
     sql """ set enable_profile=true"""
     sql """ set enable_pipeline_x_engine=true;"""

diff --git a/...test/suites/inverted_index_p0/array_contains/test_index_compaction_unique_keys_arr.groovy b/...test/suites/inverted_index_p0/array_contains/test_index_compaction_unique_keys_arr.groovy
@@ -17,7 +17,7 @@
 
 import org.codehaus.groovy.runtime.IOGroovyMethods
 
-suite("test_index_compaction_unique_keys_arr", "array_contains_inverted_index") {
+suite("test_index_compaction_unique_keys_arr", "array_contains_inverted_index, nonConcurrent") {
     // here some variable to control inverted index query
     sql """ set enable_profile=true"""
     sql """ set enable_pipeline_x_engine=true;"""

diff --git a/...est/suites/inverted_index_p0/index_compaction/test_index_compaction_empty_segments.groovy b/...est/suites/inverted_index_p0/index_compaction/test_index_compaction_empty_segments.groovy
@@ -19,7 +19,7 @@ import java.util.concurrent.Executors
 import java.util.concurrent.TimeUnit
 import org.awaitility.Awaitility
 
-suite("test_index_compaction_empty_segments", "p0") {
+suite("test_index_compaction_empty_segments", "p0, nonConcurrent") {
 
     def compaction_table_name = "test_index_compaction_empty_segments"
     def backendId_to_backendIP = [:]
@@ -82,8 +82,9 @@ suite("test_index_compaction_empty_segments", "p0") {
         });
     }
 
-    int afterSegmentCount = 0
+
     for (def tablet in tablets) {
+        int afterSegmentCount = 0
         String tablet_id = tablet.TabletId
         (code, out, err) = curl("GET", tablet.CompactionStatus)
         logger.info("Show tablets status: code=" + code + ", out=" + out + ", err=" + err)
@@ -94,6 +95,6 @@ suite("test_index_compaction_empty_segments", "p0") {
             logger.info("rowset is: " + rowset)
             afterSegmentCount += Integer.parseInt(rowset.split(" ")[1])
         }
+        assertEquals(afterSegmentCount, 0)
     }
-    assertEquals(afterSegmentCount, 0)
 }