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

kxiao pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.0 by this push:
     new 8ffdfb62313 [Fix](regression)P0 case fail in inside pipeline on 
branch-2.0 (#26197)
8ffdfb62313 is described below

commit 8ffdfb62313326819a36d549518228fd80cb24f7
Author: zhangguoqiang <[email protected]>
AuthorDate: Wed Nov 1 17:15:57 2023 +0800

    [Fix](regression)P0 case fail in inside pipeline on branch-2.0 (#26197)
---
 .../jdbc/test_jdbc_query_mysql.out                 | 12 +++----
 .../external_table_p0/jdbc/test_jdbc_query_pg.out  |  6 ++--
 .../broker_load/test_broker_load_func.groovy       | 34 +++++++++++++++-----
 .../jdbc/test_jdbc_query_mysql.groovy              | 37 ++++++++++++----------
 .../jdbc/test_jdbc_query_pg.groovy                 | 11 +++++--
 .../external_table_p0/tvf/test_hdfs_tvf.groovy     |  4 ++-
 6 files changed, 67 insertions(+), 37 deletions(-)

diff --git 
a/regression-test/data/external_table_p0/jdbc/test_jdbc_query_mysql.out 
b/regression-test/data/external_table_p0/jdbc/test_jdbc_query_mysql.out
index de2b898688f..f92290ff60c 100644
--- a/regression-test/data/external_table_p0/jdbc/test_jdbc_query_mysql.out
+++ b/regression-test/data/external_table_p0/jdbc/test_jdbc_query_mysql.out
@@ -1104,10 +1104,10 @@ false
 \N
 
 -- !sql --
-3      1970-01-01      1970-01-01      1427427954.90   0.00    0.00    0.00    
0.00    0.00    6,7,2,9 H,G,E,A 7601.25,3683.85,2449.00,5297.81,4785.38 
4,5,6,9,A       9,7,C,5 a,b,e,d,c
-5      1970-01-01      1970-01-01      163793651.90    0.00    0.00    0.00    
0.00    0.00    7       C       6276.86 8       9       d,a,b,c,e
-6      1970-01-01      1970-01-01      113878186.90    106106702.80    0.00    
0.00    0.00    343909702.45    1,4     A,E     2099.18,9137.82 C,B     8,7     
e,d,b,c,a
-8      1970-01-01      1970-01-01      487433108.65    0.00    0.00    0.00    
0.00    0.00    9       H       2900.42 A       6       c,e,b,a,d
+3      1970-01-01      1970-01-01      1427427954.90   0.00    0.00    0.00    
0.00    0.00    2,2,2,2,2,6,6,6,6,6,6,6,6,6,6,7,7,7,7,7,9,9,9,9,9       
A,A,A,A,A,E,E,E,E,E,G,G,G,G,G,H,H,H,H,H,H,H,H,H,H       
2449.00,2449.00,2449.00,2449.00,2449.00,3683.85,3683.85,3683.85,3683.85,3683.85,4785.38,4785.38,4785.38,4785.38,4785.38,5297.81,5297.81,5297.81,5297.81,5297.81,7601.25,7601.25,7601.25,7601.25,7601.25
 A,A,A,A,A,4,4,4,4,4,5,5,5,5,5,6,6,6,6,6,9,9,9,9,9       
C,C,C,C,C,C,C,C,C,C,5,5,5,5,5,7,7,7,7,7,9,9,9,9,9       
a,a,a,a,a,b,b,b,b,b,c,c,c,c,c [...]
+5      1970-01-01      1970-01-01      163793651.90    0.00    0.00    0.00    
0.00    0.00    7,7,7,7,7       C,C,C,C,C       
6276.86,6276.86,6276.86,6276.86,6276.86 8,8,8,8,8       9,9,9,9,9       
a,b,c,d,e
+6      1970-01-01      1970-01-01      113878186.90    106106702.80    0.00    
0.00    0.00    343909702.45    1,1,1,1,1,4,4,4,4       A,A,A,A,A,E,E,E,E       
2099.18,2099.18,2099.18,2099.18,2099.18,9137.82,9137.82,9137.82,9137.82 
B,B,B,B,C,C,C,C,C       7,7,7,7,8,8,8,8,8       a,a,b,b,c,c,d,d,e
+8      1970-01-01      1970-01-01      487433108.65    0.00    0.00    0.00    
0.00    0.00    9,9,9,9,9       H,H,H,H,H       
2900.42,2900.42,2900.42,2900.42,2900.42 A,A,A,A,A       6,6,6,6,6       
a,b,c,d,e
 
 -- !sql1 --
 1025
@@ -1475,9 +1475,9 @@ true      abc     efg     2022-10-01      3.4     1       
2       4       100000  1.2     2022-10-02T12:59:01     24.000  true    abc
 1.000
 
 -- !sql95 --
+0
+1
 1
-2
-3
 
 -- !sql100 --
 1025
diff --git a/regression-test/data/external_table_p0/jdbc/test_jdbc_query_pg.out 
b/regression-test/data/external_table_p0/jdbc/test_jdbc_query_pg.out
index 14b59b54a53..06b0247814a 100644
--- a/regression-test/data/external_table_p0/jdbc/test_jdbc_query_pg.out
+++ b/regression-test/data/external_table_p0/jdbc/test_jdbc_query_pg.out
@@ -1384,9 +1384,9 @@ true      abc     def     2022-10-11      1.234   1       
2       3       2022-10-22T10:59:59     34.123  true    abc     def     2022
 1.000
 
 -- !sql95 --
-123
-123
-234
+0
+1
+2
 
 -- !sql100 --
 1026
diff --git 
a/regression-test/suites/external_table_p0/broker_load/test_broker_load_func.groovy
 
b/regression-test/suites/external_table_p0/broker_load/test_broker_load_func.groovy
index c3793c605a2..e15579a7ae9 100644
--- 
a/regression-test/suites/external_table_p0/broker_load/test_broker_load_func.groovy
+++ 
b/regression-test/suites/external_table_p0/broker_load/test_broker_load_func.groovy
@@ -25,13 +25,16 @@ suite("test_broker_load_func", 
"p0,external,hive,external_docker,external_docker
 
             String database_name = "test_broker_load_func"
             String broker_name = "hdfs"
+            def uuid = UUID.randomUUID().toString().replaceAll("-", "")
+            def test_load_label="label_test_broker_load_func_${uuid}"
+            String table_name="simple"
 
             sql """drop database if exists ${database_name}; """
             sql """create database if not exists ${database_name};"""
             sql """use ${database_name}; """
 
             sql """
-                create table simple (
+                create table ${table_name} (
                     `t_empty_string`  varchar(255) NULL COMMENT '',
                     `t_string` varchar(255) NULL COMMENT ''
                 ) engine=olap
@@ -42,10 +45,10 @@ suite("test_broker_load_func", 
"p0,external,hive,external_docker,external_docker
                 """
 
             sql """
-                LOAD LABEL ${database_name}.label_test_broker_load_func
+                LOAD LABEL ${database_name}.${test_load_label}
                 (
                     DATA 
INFILE("hdfs://${externalEnvIp}:${hdfsPort}/user/doris/preinstalled_data/csv/csv_all_types/csv_all_types")
-                    INTO TABLE `simple`
+                    INTO TABLE ${table_name}
                     COLUMNS TERMINATED BY ","
                 )
                 WITH BROKER ${broker_name}
@@ -55,11 +58,26 @@ suite("test_broker_load_func", 
"p0,external,hive,external_docker,external_docker
                 );
             """
 
-            sleep(10000)
-            
-            def res = sql """select count(*) from simple;"""
-            
-            assertEquals(10,res[0][0])
+            def check_load_result = {checklabel, testTablex ->
+                max_try_milli_secs = 60000
+                while(max_try_milli_secs) {
+                    result = sql "show load where label = '${checklabel}'"
+                    if(result[0][2] == "FINISHED") {
+                        //sql "sync"
+                        def res = sql "select count(*) from 
${database_name}.${testTablex};"
+                        assertEquals(10,res[0][0])
+                        break
+                    } else {
+                        sleep(1000) // wait 1 second every time
+                        max_try_milli_secs -= 1000
+                        if(max_try_milli_secs <= 0) {
+                            assertEquals(1, 2)
+                        }
+                    }
+                }
+            }
+
+            check_load_result.call(test_load_label, table_name)
 
         } finally {
         }
diff --git 
a/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_mysql.groovy 
b/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_mysql.groovy
index 8b3f294b61e..7964b7e9370 100644
--- a/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_mysql.groovy
+++ b/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_mysql.groovy
@@ -18,6 +18,11 @@
 suite("test_jdbc_query_mysql", 
"p0,external,mysql,external_docker,external_docker_mysql") {
 
     String enabled = context.config.otherConfigs.get("enableJdbcTest")
+    String externalEnvIp = context.config.otherConfigs.get("externalEnvIp")
+    String s3_endpoint = getS3Endpoint()
+    String bucket = getS3BucketName()
+    String driver_url = 
"http://${bucket}.${s3_endpoint}/regression/jdbc_driver/mysql-connector-java-8.0.25.jar";
+
     if (enabled != null && enabled.equalsIgnoreCase("true")) {
         String mysql_57_port = context.config.otherConfigs.get("mysql_57_port")
         String jdbcResourceMysql57 = "jdbc_resource_mysql_57_x"
@@ -25,10 +30,10 @@ suite("test_jdbc_query_mysql", 
"p0,external,mysql,external_docker,external_docke
         String exMysqlTable = "doris_ex_tb";
         String exMysqlTable1 = "doris_ex_tb1";
         String exMysqlTable2 = "doris_ex_tb2";
-       String exMysqlTypeTable = "doris_ex_type_tb";
-        String inDorisTable = "doris_in_tb";
-        String inDorisTable1 = "doris_in_tb1";
-        String inDorisTable2 = "doris_in_tb2";
+           String exMysqlTypeTable = "doris_ex_type_tb";
+        String inDorisTable = "test_jdbc_mysql_doris_in_tb";
+        String inDorisTable1 = "test_jdbc_mysql_doris_in_tb1";
+        String inDorisTable2 = "test_jdbc_mysql_doris_in_tb2";
 
         sql """drop resource if exists $jdbcResourceMysql57;"""
         sql """
@@ -37,8 +42,8 @@ suite("test_jdbc_query_mysql", 
"p0,external,mysql,external_docker,external_docke
                 "type"="jdbc",
                 "user"="root",
                 "password"="123456",
-                "jdbc_url"="jdbc:mysql://127.0.0.1:$mysql_57_port/doris_test",
-                
"driver_url"="https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/jdbc_driver/mysql-connector-java-8.0.25.jar";,
+                
"jdbc_url"="jdbc:mysql://${externalEnvIp}:$mysql_57_port/doris_test",
+                "driver_url"="${driver_url}",
                 "driver_class"="com.mysql.cj.jdbc.Driver"
             );
             """
@@ -669,22 +674,22 @@ suite("test_jdbc_query_mysql", 
"p0,external,mysql,external_docker,external_docke
               sum(case when capital_type = '2' and petty_cash_type = '2' then 
IFNULL(recharge_amount, 0.00) else 0.00 end) as petty_recharge_amount,
               sum(case when capital_type = '2' and petty_cash_type = '3' then 
IFNULL(recharge_amount, 0.00) else 0.00 end) as petty_return_amount,
               sum(case when capital_type = '3' then IFNULL(need_actual_amount, 
0.00) else 0.00 end) as return_goods_amount,
-              GROUP_CONCAT(distinct cast(supplier_id as varchar (12))) 
supplier_id_list
+              GROUP_CONCAT(cast(supplier_id as varchar (12)) order by 
supplier_id) supplier_id_list
        from tmp_media_purchase group by media_order_id),
-        t2 as (select media_order_id, GROUP_CONCAT(distinct (case 
agent_policy_type 
+        t2 as (select media_order_id, GROUP_CONCAT((case agent_policy_type 
         when '1' then 'A' when '2' then 'B' when '3' then 'C' when '4' then 
'D' when '5' then 'E' when '6' then 'F'
         when '7' then 'G' when '8' then 'H' when '9' then 'I' when '10' then 
'J' when '11' then 'K' when '12' then 'L'
-        when '13' then 'M'  else agent_policy_type end)) agent_policy_type_list
+        when '13' then 'M'  else agent_policy_type end) order by 
agent_policy_type) agent_policy_type_list
        from tmp_media_purchase group by media_order_id),
-       t3 as (select media_order_id, GROUP_CONCAT(distinct cast(agent_policy 
as varchar (12))) agent_policy_list
+       t3 as (select media_order_id, GROUP_CONCAT(cast(agent_policy as varchar 
(12)) order by agent_policy) agent_policy_list
        from tmp_media_purchase group by media_order_id),
-       t4 as (select media_order_id, GROUP_CONCAT(distinct (case capital_type
-        when '1' then 'A' when '2' then 'B' when '3' then 'C' else 
capital_type end)) capital_type_list
+       t4 as (select media_order_id, GROUP_CONCAT((case capital_type
+        when '1' then 'A' when '2' then 'B' when '3' then 'C' else 
capital_type end) order by capital_type) capital_type_list
        from tmp_media_purchase group by media_order_id),
-       t5 as (select media_order_id, GROUP_CONCAT(distinct (case 
petty_cash_type
-        when '1' then 'A' when '2' then 'B' when '3' then 'C' else 
petty_cash_type end)) petty_cash_type_list
+       t5 as (select media_order_id, GROUP_CONCAT((case petty_cash_type
+        when '1' then 'A' when '2' then 'B' when '3' then 'C' else 
petty_cash_type end) order by petty_cash_type) petty_cash_type_list
        from tmp_media_purchase group by media_order_id),
-       t6 as (select media_order_id, GROUP_CONCAT(distinct `name`) 
company_name_list
+       t6 as (select media_order_id, GROUP_CONCAT(`name` order by `name`) 
company_name_list
             from tmp_media_purchase group by media_order_id)
         select distinct tmp_media_purchase.`media_order_id`,
                 first_payment_date,
@@ -913,7 +918,7 @@ suite("test_jdbc_query_mysql", 
"p0,external,mysql,external_docker,external_docke
         order_qt_sql93 """ SELECT CASE k8 WHEN 1 THEN CAST(1 AS decimal(4,1)) 
WHEN 2 THEN CAST(1 AS decimal(4,2)) 
                             ELSE CAST(1 AS decimal(4,3)) END FROM 
$jdbcMysql57Table1 limit 3"""
         order_qt_sql95 """ SELECT * from (SELECT k8 FROM $jdbcMysql57Table1 
UNION (SELECT id as k8 FROM ${exMysqlTable}  UNION SELECT k7 as k8 FROM 
$jdbcMysql57Table1) 
-                            UNION ALL SELECT products_id as k8 FROM 
$exMysqlTable1 ORDER BY k8 limit 3) as a  limit 3"""
+                            UNION ALL SELECT products_id as k8 FROM 
$exMysqlTable1 ORDER BY k8 limit 3) as a  order by k8 limit 3"""
         order_qt_sql100 """ SELECT COUNT(*) FROM $jdbcMysql57Table1 WHERE 
EXISTS(SELECT max(id) FROM ${exMysqlTable}) """
         order_qt_sql103 """ SELECT count(*) FROM $jdbcMysql57Table1 n WHERE 
(SELECT count(*) FROM ${exMysqlTable} r WHERE n.k8 = r.id) > 1 """
         order_qt_sql105 """ SELECT count(*) AS numwait FROM $jdbcMysql57Table1 
l1 WHERE
diff --git 
a/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_pg.groovy 
b/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_pg.groovy
index 7a41885d7cc..7eabc901f44 100644
--- a/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_pg.groovy
+++ b/regression-test/suites/external_table_p0/jdbc/test_jdbc_query_pg.groovy
@@ -20,6 +20,11 @@ import java.nio.charset.Charset;
 suite("test_jdbc_query_pg", 
"p0,external,pg,external_docker,external_docker_pg") {
 
     String enabled = context.config.otherConfigs.get("enableJdbcTest")
+    String externalEnvIp = context.config.otherConfigs.get("externalEnvIp")
+    String s3_endpoint = getS3Endpoint()
+    String bucket = getS3BucketName()
+    String driver_url = 
"https://${bucket}.${s3_endpoint}/regression/jdbc_driver/postgresql-42.5.0.jar";
+
     if (enabled != null && enabled.equalsIgnoreCase("true")) {
         String pg_14_port = context.config.otherConfigs.get("pg_14_port")
         String jdbcResourcePg14 = "jdbc_resource_pg_14"
@@ -43,8 +48,8 @@ suite("test_jdbc_query_pg", 
"p0,external,pg,external_docker,external_docker_pg")
                 "type"="jdbc",
                 "user"="postgres",
                 "password"="123456",
-                
"jdbc_url"="jdbc:postgresql://127.0.0.1:$pg_14_port/postgres?currentSchema=doris_test",
-                
"driver_url"="https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/jdbc_driver/postgresql-42.5.0.jar";,
+                
"jdbc_url"="jdbc:postgresql://${externalEnvIp}:$pg_14_port/postgres?currentSchema=doris_test",
+                "driver_url"="${driver_url}",
                 "driver_class"="org.postgresql.Driver"
             );
             """
@@ -616,7 +621,7 @@ suite("test_jdbc_query_pg", 
"p0,external,pg,external_docker,external_docker_pg")
         order_qt_sql93 """ SELECT CASE k8 WHEN 1 THEN CAST(1 AS decimal(4,1)) 
WHEN 2 THEN CAST(1 AS decimal(4,2)) 
                             ELSE CAST(1 AS decimal(4,3)) END FROM 
$jdbcPg14Table1 limit 3"""
         order_qt_sql95 """ SELECT * from (SELECT k8 FROM $jdbcPg14Table1 UNION 
(SELECT id as k8 FROM ${dorisExTable1}  UNION SELECT k7 as k8 FROM 
$jdbcPg14Table1) 
-                            UNION ALL SELECT id as k8 FROM $exMysqlTypeTable 
ORDER BY id limit 3) as a  limit 3"""
+                            UNION ALL SELECT id as k8 FROM $exMysqlTypeTable 
ORDER BY id limit 3) as a order by k8 limit 3"""
         order_qt_sql100 """ SELECT COUNT(*) FROM $jdbcPg14Table1 WHERE 
EXISTS(SELECT max(id) FROM ${dorisExTable1}) """
         order_qt_sql103 """ SELECT count(*) FROM $jdbcPg14Table1 n WHERE 
(SELECT count(*) FROM ${dorisExTable1} r WHERE n.k8 = r.id) > 1 """
         order_qt_sql105 """ SELECT count(*) AS numwait FROM $jdbcPg14Table1 l1 
WHERE
diff --git a/regression-test/suites/external_table_p0/tvf/test_hdfs_tvf.groovy 
b/regression-test/suites/external_table_p0/tvf/test_hdfs_tvf.groovy
index 856646fff21..61c049a0a04 100644
--- a/regression-test/suites/external_table_p0/tvf/test_hdfs_tvf.groovy
+++ b/regression-test/suites/external_table_p0/tvf/test_hdfs_tvf.groovy
@@ -17,10 +17,12 @@
 
 suite("test_hdfs_tvf","external,hive,tvf,external_docker") {
     String hdfs_port = context.config.otherConfigs.get("hdfs_port")
+    String externalEnvIp = context.config.otherConfigs.get("externalEnvIp")
+
     // It's okay to use random `hdfsUser`, but can not be empty.
     def hdfsUserName = "doris"
     def format = "csv"
-    def defaultFS = "hdfs://127.0.0.1:${hdfs_port}"
+    def defaultFS = "hdfs://${externalEnvIp}:${hdfs_port}"
     def uri = ""
 
     String enabled = context.config.otherConfigs.get("enableHiveTest")


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

Reply via email to