[Test] improve test cases stablility

shenyunlong · shenyunlong · commit f5d91c6d11c1 · 2025-03-26T18:39:02.000+08:00
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartAppendTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartAppendTest.java
@@ -55,7 +55,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-//        dropTables(tableNames, group2tableNames);
     }
 
     @Before
@@ -270,6 +269,5 @@ public void testAppendMultiCF() throws Throwable {
     public void testAppendSeires() throws Throwable {
         createTables(TableTemplateManager.TableType.SECONDARY_PARTITIONED_TIME_RANGE_KEY, series_tables, null, true);
         FOR_EACH(series_tables, OHTableSecondaryPartAppendTest::testAppendSeires);
-        dropTables(series_tables, null);
     }
 }
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartBatchGetTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartBatchGetTest.java
@@ -50,7 +50,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartBatchTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartBatchTest.java
@@ -52,7 +52,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartCheckAndMutateTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartCheckAndMutateTest.java
@@ -57,7 +57,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
@@ -441,6 +440,5 @@ public void testCheckAndMutateSeires() throws Throwable {
         Map<String, List<String>> seriesTablesMultiCF = new LinkedHashMap<String, List<String>>();
         createTables(TableTemplateManager.TableType.SECONDARY_PARTITIONED_TIME_RANGE_KEY, seriesTables, seriesTablesMultiCF, true);
         FOR_EACH(seriesTables, OHTableSecondaryPartCheckAndMutateTest::testCheckAndMutateSeires);
-        dropTables(seriesTables, seriesTablesMultiCF);
     }
 }
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartDeleteTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartDeleteTest.java
@@ -48,7 +48,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartGetTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartGetTest.java
@@ -57,7 +57,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartIncrementTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartIncrementTest.java
@@ -54,7 +54,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
@@ -262,6 +261,5 @@ public void testIncrementSeires() throws Throwable {
         List<String> series_tables = new LinkedList<String>();
         createTables(TableTemplateManager.TableType.SECONDARY_PARTITIONED_TIME_RANGE_KEY, series_tables, null, true);
         FOR_EACH(series_tables, OHTableSecondaryPartIncrementTest::testIncrementSeires);
-        dropTables(series_tables, null);
     }
 }
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartPutTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartPutTest.java
@@ -48,7 +48,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartScanTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableSecondaryPartScanTest.java
@@ -49,7 +49,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableTimeSeriesGetTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableTimeSeriesGetTest.java
@@ -55,7 +55,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableTimeSeriesScanTest.java b/src/test/java/com/alipay/oceanbase/hbase/secondary/OHTableTimeSeriesScanTest.java
@@ -48,7 +48,6 @@ public static void before() throws Exception {
     @AfterClass
     public static void finish() throws Exception {
         closeDistributedExecute();
-        dropTables(tableNames, group2tableNames);
     }
 
     @Before
diff --git a/src/test/java/com/alipay/oceanbase/hbase/util/TableTemplateManager.java b/src/test/java/com/alipay/oceanbase/hbase/util/TableTemplateManager.java
@@ -84,8 +84,6 @@ public enum TableType {
     private static final Map<TableType, String> SQL_TEMPLATES            = new EnumMap<TableType, String>(
                                                                              TableType.class);
 
-    private static final Map<TableType, String> SQL_TEMPLATES = new EnumMap<>(TableType.class);
-
     static {
         // 普通表非分区表模版
         SQL_TEMPLATES.put(TableType.NON_PARTITIONED_REGULAR,

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,6 @@ public static void before() throws Exception {`
`55`	`55`	`@AfterClass`
`56`	`56`	`public static void finish() throws Exception {`
`57`	`57`	`closeDistributedExecute();`
`58`		`-// dropTables(tableNames, group2tableNames);`
`59`	`58`	`}`
`60`	`59`
`61`	`60`	`@Before`
`@@ -270,6 +269,5 @@ public void testAppendMultiCF() throws Throwable {`
`270`	`269`	`public void testAppendSeires() throws Throwable {`
`271`	`270`	`createTables(TableTemplateManager.TableType.SECONDARY_PARTITIONED_TIME_RANGE_KEY, series_tables, null, true);`
`272`	`271`	`FOR_EACH(series_tables, OHTableSecondaryPartAppendTest::testAppendSeires);`
`273`		`- dropTables(series_tables, null);`
`274`	`272`	`}`
`275`	`273`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,6 @@ public static void before() throws Exception {`
`50`	`50`	`@AfterClass`
`51`	`51`	`public static void finish() throws Exception {`
`52`	`52`	`closeDistributedExecute();`
`53`		`- dropTables(tableNames, group2tableNames);`
`54`	`53`	`}`
`55`	`54`
`56`	`55`	`@Before`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,6 @@ public static void before() throws Exception {`
`52`	`52`	`@AfterClass`
`53`	`53`	`public static void finish() throws Exception {`
`54`	`54`	`closeDistributedExecute();`
`55`		`- dropTables(tableNames, group2tableNames);`
`56`	`55`	`}`
`57`	`56`
`58`	`57`	`@Before`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,6 @@ public static void before() throws Exception {`
`57`	`57`	`@AfterClass`
`58`	`58`	`public static void finish() throws Exception {`
`59`	`59`	`closeDistributedExecute();`
`60`		`- dropTables(tableNames, group2tableNames);`
`61`	`60`	`}`
`62`	`61`
`63`	`62`	`@Before`
`@@ -441,6 +440,5 @@ public void testCheckAndMutateSeires() throws Throwable {`
`441`	`440`	`Map<String, List<String>> seriesTablesMultiCF = new LinkedHashMap<String, List<String>>();`
`442`	`441`	`createTables(TableTemplateManager.TableType.SECONDARY_PARTITIONED_TIME_RANGE_KEY, seriesTables, seriesTablesMultiCF, true);`
`443`	`442`	`FOR_EACH(seriesTables, OHTableSecondaryPartCheckAndMutateTest::testCheckAndMutateSeires);`
`444`		`- dropTables(seriesTables, seriesTablesMultiCF);`
`445`	`443`	`}`
`446`	`444`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,6 @@ public static void before() throws Exception {`
`48`	`48`	`@AfterClass`
`49`	`49`	`public static void finish() throws Exception {`
`50`	`50`	`closeDistributedExecute();`
`51`		`- dropTables(tableNames, group2tableNames);`
`52`	`51`	`}`
`53`	`52`
`54`	`53`	`@Before`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,6 @@ public static void before() throws Exception {`
`54`	`54`	`@AfterClass`
`55`	`55`	`public static void finish() throws Exception {`
`56`	`56`	`closeDistributedExecute();`
`57`		`- dropTables(tableNames, group2tableNames);`
`58`	`57`	`}`
`59`	`58`
`60`	`59`	`@Before`
`@@ -262,6 +261,5 @@ public void testIncrementSeires() throws Throwable {`
`262`	`261`	`List<String> series_tables = new LinkedList<String>();`
`263`	`262`	`createTables(TableTemplateManager.TableType.SECONDARY_PARTITIONED_TIME_RANGE_KEY, series_tables, null, true);`
`264`	`263`	`FOR_EACH(series_tables, OHTableSecondaryPartIncrementTest::testIncrementSeires);`
`265`		`- dropTables(series_tables, null);`
`266`	`264`	`}`
`267`	`265`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,6 @@ public static void before() throws Exception {`
`49`	`49`	`@AfterClass`
`50`	`50`	`public static void finish() throws Exception {`
`51`	`51`	`closeDistributedExecute();`
`52`		`- dropTables(tableNames, group2tableNames);`
`53`	`52`	`}`
`54`	`53`
`55`	`54`	`@Before`