[fix](regression) Fix some p0 case (apache#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 apache#45211
shuke987 · Dec 23, 2024 · 3be7eb4 · 3be7eb4
1 parent b68edac
commit 3be7eb4
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/...t/suites/unique_with_mow_c_p0/partial_update/test_partial_update_2pc_schema_change.groovy b/...t/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
@@ -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/...est/suites/unique_with_mow_p0/partial_update/test_partial_update_2pc_schema_change.groovy b/...est/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"