From 0e85e183d142bd64769349efaef83bb167576c01 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 24 Feb 2025 16:36:45 +0800 Subject: [PATCH] branch-2.1: [test](external) Add `restart_fe` tag to test suites #48181 (#48235) Cherry-picked from #48181 Co-authored-by: zy-kkk --- .../lower_case/upgrade/load.groovy | 25 +------------------ .../test_upgrade_lower_case_catalog.groovy | 2 +- 2 files changed, 2 insertions(+), 25 deletions(-) diff --git a/regression-test/suites/external_table_p0/lower_case/upgrade/load.groovy b/regression-test/suites/external_table_p0/lower_case/upgrade/load.groovy index 782f7b3711..5123817f4f 100644 --- a/regression-test/suites/external_table_p0/lower_case/upgrade/load.groovy +++ b/regression-test/suites/external_table_p0/lower_case/upgrade/load.groovy @@ -19,7 +19,7 @@ import java.util.concurrent.Executors import java.util.concurrent.TimeUnit import org.awaitility.Awaitility -suite("test_upgrade_lower_case_catalog_prepare", "p0,external,doris,external_docker,external_docker_doris") { +suite("test_upgrade_lower_case_catalog_prepare", "p0,external,doris,external_docker,external_docker_doris,restart_fe") { String jdbcUrl = context.config.jdbcUrl String jdbcUser = "test_upgrade_lower_case_catalog_user" @@ -80,29 +80,6 @@ suite("test_upgrade_lower_case_catalog_prepare", "p0,external,doris,external_doc )""" wait_table_sync("test_upgrade_lower_case_catalog.upgrade_lower_case_catalog_lower") - test { - sql """show databases from test_upgrade_lower_case_catalog""" - - // Verification results include external_test_lower and external_test_UPPER - check { result, ex, startTime, endTime -> - def expectedDatabases = ["upgrade_lower_case_catalog_lower", "upgrade_lower_case_catalog_upper"] - expectedDatabases.each { dbName -> - assertTrue(result.collect { it[0] }.contains(dbName), "Expected database '${dbName}' not found in result") - } - } - } - - test { - sql """show tables from test_upgrade_lower_case_catalog.upgrade_lower_case_catalog_lower""" - - // Verification results include lower and UPPER - check { result, ex, startTime, endTime -> - def expectedTables = ["lower", "upper"] - expectedTables.each { tableName -> - assertTrue(result.collect { it[0] }.contains(tableName), "Expected table '${tableName}' not found in result") - } - } - } qt_sql_test_upgrade_lower_case_catalog_1 "select * from test_upgrade_lower_case_catalog.upgrade_lower_case_catalog_lower.lower" qt_sql_test_upgrade_lower_case_catalog_2 "select * from test_upgrade_lower_case_catalog.upgrade_lower_case_catalog_lower.upper" diff --git a/regression-test/suites/external_table_p0/lower_case/upgrade/test_upgrade_lower_case_catalog.groovy b/regression-test/suites/external_table_p0/lower_case/upgrade/test_upgrade_lower_case_catalog.groovy index 634e5e5737..9b3c3433ce 100644 --- a/regression-test/suites/external_table_p0/lower_case/upgrade/test_upgrade_lower_case_catalog.groovy +++ b/regression-test/suites/external_table_p0/lower_case/upgrade/test_upgrade_lower_case_catalog.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_upgrade_lower_case_catalog", "p0,external,doris,external_docker,external_docker_doris") { +suite("test_upgrade_lower_case_catalog", "p0,external,doris,external_docker,external_docker_doris,restart_fe") { test { sql """show databases from test_upgrade_lower_case_catalog"""