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