From 5de3302b227ded4b0060165b63d87cd6974a0dfd Mon Sep 17 00:00:00 2001 From: zgxme Date: Thu, 22 Oct 2026 15:15:47 +0800 Subject: [PATCH] [test](regression) Run isolated external suites concurrently (#69310) Problem Summary: The external regression pipeline schedules every suite tagged nonConcurrent through a single-thread executor, even when the suite has an isolated catalog, database, and table namespace. Six Paimon Variant write suites and the Iceberg initial-default suite do not mutate FE or BE global configuration, inject debug points, or assert process-global metrics. Remove their nonConcurrent tag so the existing external suite parallelism can schedule them with other isolated suites while retaining serial execution for suites that require exclusive cluster state. --- .../iceberg/test_iceberg_initial_defaults.groovy | 2 +- .../paimon/write/test_paimon_write_variant.groovy | 2 +- .../paimon/write/test_paimon_write_variant_dml.groovy | 2 +- .../paimon/write/test_paimon_write_variant_errors.groovy | 2 +- .../paimon/write/test_paimon_write_variant_nested.groovy | 2 +- .../paimon/write/test_paimon_write_variant_shredding.groovy | 2 +- .../paimon/write/test_paimon_write_variant_table_modes.groovy | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/regression-test/suites/external_table_p0/iceberg/test_iceberg_initial_defaults.groovy b/regression-test/suites/external_table_p0/iceberg/test_iceberg_initial_defaults.groovy index 14852d1756997c..c80590e645c8c4 100644 --- a/regression-test/suites/external_table_p0/iceberg/test_iceberg_initial_defaults.groovy +++ b/regression-test/suites/external_table_p0/iceberg/test_iceberg_initial_defaults.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_iceberg_initial_defaults", "p0,external,nonConcurrent") { +suite("test_iceberg_initial_defaults", "p0,external") { String enabled = context.config.otherConfigs.get("enableIcebergTest") if (enabled == null || !enabled.equalsIgnoreCase("true")) { logger.info("disable iceberg test.") diff --git a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant.groovy b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant.groovy index 5a6818ba4b96e2..dd10c59c94fa65 100644 --- a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant.groovy +++ b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_paimon_write_variant", "p0,external,paimon,nonConcurrent") { +suite("test_paimon_write_variant", "p0,external,paimon") { String enabled = context.config.otherConfigs.get("enablePaimonTest") if (enabled == null || !enabled.equalsIgnoreCase("true")) { logger.info("disable paimon test.") diff --git a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_dml.groovy b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_dml.groovy index 19ab351a83f767..565ff8aa6af78d 100644 --- a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_dml.groovy +++ b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_dml.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_paimon_write_variant_dml", "p0,external,paimon,nonConcurrent") { +suite("test_paimon_write_variant_dml", "p0,external,paimon") { String enabled = context.config.otherConfigs.get("enablePaimonTest") if (enabled == null || !enabled.equalsIgnoreCase("true")) { logger.info("disable paimon test.") diff --git a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_errors.groovy b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_errors.groovy index 88df82ebe09d97..3c95bacbb87cd7 100644 --- a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_errors.groovy +++ b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_errors.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_paimon_write_variant_errors", "p0,external,paimon,nonConcurrent") { +suite("test_paimon_write_variant_errors", "p0,external,paimon") { String enabled = context.config.otherConfigs.get("enablePaimonTest") if (enabled == null || !enabled.equalsIgnoreCase("true")) { logger.info("disable paimon test.") diff --git a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_nested.groovy b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_nested.groovy index 2e883a63dc5b44..41cf1d005c8c70 100644 --- a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_nested.groovy +++ b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_nested.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_paimon_write_variant_nested", "p0,external,paimon,nonConcurrent") { +suite("test_paimon_write_variant_nested", "p0,external,paimon") { String enabled = context.config.otherConfigs.get("enablePaimonTest") if (enabled == null || !enabled.equalsIgnoreCase("true")) { logger.info("disable paimon test.") diff --git a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_shredding.groovy b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_shredding.groovy index 7856f9cbbb4948..aefb520f82768b 100644 --- a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_shredding.groovy +++ b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_shredding.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_paimon_write_variant_shredding", "p0,external,paimon,nonConcurrent") { +suite("test_paimon_write_variant_shredding", "p0,external,paimon") { String enabled = context.config.otherConfigs.get("enablePaimonTest") if (enabled == null || !enabled.equalsIgnoreCase("true")) { logger.info("disable paimon test.") diff --git a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_table_modes.groovy b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_table_modes.groovy index 35d78c42f26389..9ede6133d2e505 100644 --- a/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_table_modes.groovy +++ b/regression-test/suites/external_table_p0/paimon/write/test_paimon_write_variant_table_modes.groovy @@ -15,7 +15,7 @@ // specific language governing permissions and limitations // under the License. -suite("test_paimon_write_variant_table_modes", "p0,external,paimon,nonConcurrent") { +suite("test_paimon_write_variant_table_modes", "p0,external,paimon") { String enabled = context.config.otherConfigs.get("enablePaimonTest") if (enabled == null || !enabled.equalsIgnoreCase("true")) { logger.info("disable paimon test.")