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

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


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new 627d20d9c2e branch-3.0: [fix](regression-test) fix injection would not 
be removed when exception #46357 (#46359)
627d20d9c2e is described below

commit 627d20d9c2e6d7f72b063fbc5cca69a67247d7cb
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Mon Jan 13 21:35:59 2025 +0800

    branch-3.0: [fix](regression-test) fix injection would not be removed when 
exception #46357 (#46359)
    
    Cherry-picked from #46357
    
    Co-authored-by: hui lai <[email protected]>
---
 regression-test/README.md                                               | 2 ++
 .../suites/load_p0/routine_load/test_multi_table_load_error.groovy      | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/regression-test/README.md b/regression-test/README.md
index 1cc6aca452e..fb7bdde2ee2 100644
--- a/regression-test/README.md
+++ b/regression-test/README.md
@@ -82,6 +82,8 @@ under the License.
 
 7. Do not create the same table in different cases under the same directory to 
avoid conflicts.
 
+8. Cases injected should be marked as nonConcurrent and ensured injection to 
be removed after running the case.
+
 ## Compatibility case
 
 Refers to the resources or rules created on the initial cluster during FE 
testing or upgrade testing, which can still be used normally after the cluster 
restart or upgrade, such as permissions, UDF, etc.
diff --git 
a/regression-test/suites/load_p0/routine_load/test_multi_table_load_error.groovy
 
b/regression-test/suites/load_p0/routine_load/test_multi_table_load_error.groovy
index 7217109c353..17a8e5da719 100644
--- 
a/regression-test/suites/load_p0/routine_load/test_multi_table_load_error.groovy
+++ 
b/regression-test/suites/load_p0/routine_load/test_multi_table_load_error.groovy
@@ -125,7 +125,6 @@ suite("test_multi_table_load_eror","nonConcurrent") {
                     if (state == "RUNNING") {
                         count++
                         if (count > 60) {
-                            
GetDebugPoint().disableDebugPointForAllBEs(injection)
                             break;
                         }
                         continue;
@@ -150,6 +149,7 @@ suite("test_multi_table_load_eror","nonConcurrent") {
                     break;
                 }
             } finally {
+                GetDebugPoint().disableDebugPointForAllBEs(injection)
                 sql "stop routine load for ${jobName}"
                 sql "DROP TABLE IF EXISTS ${tableName}"
             }


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

Reply via email to