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

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


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

commit 69de40ca271851a5d6e72e88c2e2886c6446f4ce
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Jan 3 22:02:50 2025 +0800

    branch-2.1: [fix](regression-test) fix injection would not be removed when 
exception #46357 (#46360)
    
    Cherry-picked from #46357
    
    Co-authored-by: hui lai <lai...@selectdb.com>
---
 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: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to