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 15673516a94 branch-3.0: [test](move-memtable) fix false positives in 
sinkv2 injection tests #44552 (#44930)
15673516a94 is described below

commit 15673516a948f8d1f9cf58b45f91230abf40b3a5
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Dec 3 21:59:38 2024 +0800

    branch-3.0: [test](move-memtable) fix false positives in sinkv2 injection 
tests #44552 (#44930)
    
    Cherry-picked from #44552
    
    Co-authored-by: Kaijie Chen <[email protected]>
---
 .../suites/fault_injection_p0/test_writer_v2_fault_injection.groovy  | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/regression-test/suites/fault_injection_p0/test_writer_v2_fault_injection.groovy
 
b/regression-test/suites/fault_injection_p0/test_writer_v2_fault_injection.groovy
index f05a084035c..7e71de173e9 100644
--- 
a/regression-test/suites/fault_injection_p0/test_writer_v2_fault_injection.groovy
+++ 
b/regression-test/suites/fault_injection_p0/test_writer_v2_fault_injection.groovy
@@ -67,10 +67,11 @@ suite("test_writer_v2_fault_injection", "nonConcurrent") {
         file "baseall.txt"
     }
 
-    def load_with_injection = { injection, error_msg->
+    def load_with_injection = { injection, error_msg, success=false->
         try {
             GetDebugPoint().enableDebugPointForAllBEs(injection)
             sql "insert into test select * from baseall where k1 <= 3"
+            assertTrue(success, String.format("expected Exception '%s', actual 
success", error_msg))
         } catch(Exception e) {
             logger.info(e.getMessage())
             assertTrue(e.getMessage().contains(error_msg),
@@ -88,7 +89,7 @@ suite("test_writer_v2_fault_injection", "nonConcurrent") {
     // VTabletWriterV2 node_info is null
     
load_with_injection("VTabletWriterV2._open_streams_to_backend.node_info_null", 
"failed to open streams to any BE")
     // VTabletWriterV2 do not get tablet schema on open_streams
-    
load_with_injection("VTabletWriterV2._open_streams_to_backend.no_schema_when_open_streams",
 "success")
+    
load_with_injection("VTabletWriterV2._open_streams_to_backend.no_schema_when_open_streams",
 "success", true)
     // VTabletWriterV2 tablet_location is null
     
load_with_injection("VTabletWriterV2._build_tablet_node_mapping.tablet_location_null",
 "unknown tablet location")
     // VTabletWriterV2 location is null


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

Reply via email to