This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new d9aaf641226 [fix](test) fix unstable external p0 tests (#42158)
d9aaf641226 is described below

commit d9aaf641226369b5d62f147cad228f0cd3d389a6
Author: Rayner Chen <[email protected]>
AuthorDate: Mon Oct 21 16:29:09 2024 +0800

    [fix](test) fix unstable external p0 tests (#42158)
    
    ### What problem does this PR solve?
    
    Problem Summary:
    1. `test_jdbc_catalog_ddl` use same catalog name with
    `test_mysql_jdbc5_catalog`, which cause concurrency issue.
    2. comment out `paimon_base_filesystem` test because cos env is not
    ready
    
    - Release note
    
        None
---
 .../suites/external_table_p0/jdbc/test_jdbc_catalog_ddl.groovy        | 2 +-
 .../suites/external_table_p0/paimon/paimon_base_filesystem.groovy     | 4 +++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/regression-test/suites/external_table_p0/jdbc/test_jdbc_catalog_ddl.groovy 
b/regression-test/suites/external_table_p0/jdbc/test_jdbc_catalog_ddl.groovy
index 1b25362edd7..c31a98895fb 100644
--- a/regression-test/suites/external_table_p0/jdbc/test_jdbc_catalog_ddl.groovy
+++ b/regression-test/suites/external_table_p0/jdbc/test_jdbc_catalog_ddl.groovy
@@ -25,7 +25,7 @@ suite("test_jdbc_catalog_ddl", 
"p0,external,mysql,external_docker,external_docke
     String mysql_port = context.config.otherConfigs.get("mysql_57_port");
     // String driver_url = "mysql-connector-java-5.1.49.jar"
     if (enabled != null && enabled.equalsIgnoreCase("true")) {
-        String catalog_name = "mysql_jdbc5_catalog";
+        String catalog_name = "test_jdbc_catalog_ddl";
 
         for (String useMetaCache : ["true", "false"]) {
             sql """drop catalog if exists ${catalog_name} """
diff --git 
a/regression-test/suites/external_table_p0/paimon/paimon_base_filesystem.groovy 
b/regression-test/suites/external_table_p0/paimon/paimon_base_filesystem.groovy
index e4adec6b7ab..1e4561c4065 100644
--- 
a/regression-test/suites/external_table_p0/paimon/paimon_base_filesystem.groovy
+++ 
b/regression-test/suites/external_table_p0/paimon/paimon_base_filesystem.groovy
@@ -17,7 +17,9 @@
 
 suite("paimon_base_filesystem", 
"p0,external,doris,external_docker,external_docker_doris") {
     String enabled = context.config.otherConfigs.get("enablePaimonTest")
-    if (enabled == null || !enabled.equalsIgnoreCase("true")) {
+    // if (enabled == null || !enabled.equalsIgnoreCase("true")) {
+    if (true) {
+        // temporary comment out, will add back when env is ready
         return
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to