From ccac4208e1fae21743765ccd39ef49c0fc42f18c Mon Sep 17 00:00:00 2001 From: yujun Date: Tue, 20 Feb 2024 22:22:21 +0800 Subject: [PATCH] [fix](regression test) fix test_dynamic_partition_with_alter #31154 --- .../test_dynamic_partition_with_alter.groovy | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/regression-test/suites/partition_p0/dynamic_partition/test_dynamic_partition_with_alter.groovy b/regression-test/suites/partition_p0/dynamic_partition/test_dynamic_partition_with_alter.groovy index a844ad7549..d488a53812 100644 --- a/regression-test/suites/partition_p0/dynamic_partition/test_dynamic_partition_with_alter.groovy +++ b/regression-test/suites/partition_p0/dynamic_partition/test_dynamic_partition_with_alter.groovy @@ -38,14 +38,19 @@ suite("test_dynamic_partition_with_alter") { result = sql "show partitions from ${tbl}" assertEquals(7, result.size()) - test { - sql "alter table ${tbl} set ('dynamic_partition.start' = '-2147483648')" - exception "Provide start or history_partition_num property" - } + try { + test { + sql "alter table ${tbl} set ('dynamic_partition.start' = '-2147483648')" + exception "Provide start or history_partition_num property" + } - test { - sql "alter table ${tbl} set ('dynamic_partition.start' = '-2147483647')" - exception "Too many dynamic partitions" + test { + sql "alter table ${tbl} set ('dynamic_partition.start' = '-2147483647')" + exception "Too many dynamic partitions" + } + } catch (Exception e) { + sql "drop table if exists ${tbl}" + throw e } // modify distributed column comment, then try to add too more dynamic partition