(doris) branch master updated: [fix](regression) Fix some p0 case (#45333)

[hellostephen]

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

hellostephen 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 069a33a2fac [fix](regression) Fix some p0 case (#45333)
069a33a2fac is described below

commit 069a33a2faca6dbfa58800698c1257a42a6e6c25
Author: bobhan1 <bao...@selectdb.com>
AuthorDate: Thu Dec 12 14:52:56 2024 +0800

    [fix](regression) Fix some p0 case (#45333)
    
    ### What problem does this PR solve?
    
    
    fix `test_partial_update_2pc_schema_change` and
    `test_f_2pc_schema_change` case due to changes in
    https://github.com/apache/doris/pull/45211
---
 .../partial_update/test_partial_update_2pc_schema_change.groovy      | 5 +++--
 .../flexible/legacy/test_f_2pc_schema_change.groovy                  | 5 +++--
 .../partial_update/test_partial_update_2pc_schema_change.groovy      | 5 +++--
 3 files changed, 9 insertions(+), 6 deletions(-)

diff --git 
a/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_2pc_schema_change.groovy
 
b/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_2pc_schema_change.groovy
index e7205620ef3..6b3c4572dab 100644
--- 
a/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_2pc_schema_change.groovy
+++ 
b/regression-test/suites/unique_with_mow_c_p0/partial_update/test_partial_update_2pc_schema_change.groovy
@@ -38,11 +38,12 @@ suite("test_partial_update_2pc_schema_change", "p0") {
 
     String db = context.config.getDbNameByFile(context.file)
     sql "select 1;" // to create database
-
+    def user = context.config.jdbcUser
+    def password = context.config.jdbcPassword
     for (def use_row_store : [false, true]) {
         logger.info("current params: use_row_store: ${use_row_store}")
 
-        connect( context.config.jdbcUser, context.config.jdbcPassword, 
context.config.jdbcUrl) {
+        connect( user, password, context.config.jdbcUrl) {
             sql "use ${db};"
 
             def tableName = "test_partial_update_2pc_schema_change"
diff --git 
a/regression-test/suites/unique_with_mow_p0/flexible/legacy/test_f_2pc_schema_change.groovy
 
b/regression-test/suites/unique_with_mow_p0/flexible/legacy/test_f_2pc_schema_change.groovy
index 23a9dce1001..776fad1da78 100644
--- 
a/regression-test/suites/unique_with_mow_p0/flexible/legacy/test_f_2pc_schema_change.groovy
+++ 
b/regression-test/suites/unique_with_mow_p0/flexible/legacy/test_f_2pc_schema_change.groovy
@@ -37,14 +37,15 @@ import org.apache.http.util.EntityUtils
 suite("test_f_2pc_schema_change", "p0") {
     String db = context.config.getDbNameByFile(context.file)
     sql "select 1;" // to create database
-
+    def user = context.config.jdbcUser
+    def password = context.config.jdbcPassword
     if (isCloudMode()) {
         return
     }
 
     for (def use_row_store : [false, true]) {
         logger.info("current params: use_row_store: ${use_row_store}")
-        connect( context.config.jdbcUser, context.config.jdbcPassword, 
context.config.jdbcUrl) {
+        connect( user, password, context.config.jdbcUrl) {
             sql "use ${db};"
 
             def tableName = "test_f_2pc_schema_change"
diff --git 
a/regression-test/suites/unique_with_mow_p0/partial_update/test_partial_update_2pc_schema_change.groovy
 
b/regression-test/suites/unique_with_mow_p0/partial_update/test_partial_update_2pc_schema_change.groovy
index 9b323dec19c..d506d0ee766 100644
--- 
a/regression-test/suites/unique_with_mow_p0/partial_update/test_partial_update_2pc_schema_change.groovy
+++ 
b/regression-test/suites/unique_with_mow_p0/partial_update/test_partial_update_2pc_schema_change.groovy
@@ -38,11 +38,12 @@ suite("test_partial_update_2pc_schema_change", "p0") {
 
     String db = context.config.getDbNameByFile(context.file)
     sql "select 1;" // to create database
-
+    def user = context.config.jdbcUser
+    def password = context.config.jdbcPassword
     for (def use_row_store : [false, true]) {
         logger.info("current params: use_row_store: ${use_row_store}")
 
-        connect( context.config.jdbcUser, context.config.jdbcPassword, 
context.config.jdbcUrl) {
+        connect( user, password, context.config.jdbcUrl) {
             sql "use ${db};"
 
             def tableName = "test_partial_update_2pc_schema_change"


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org