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

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


The following commit(s) were added to refs/heads/branch-4.1 by this push:
     new 6edfdcf96e3 [fix](regression) fix full compaction run status case 
(#65360)
6edfdcf96e3 is described below

commit 6edfdcf96e3cbcf7fea611062a4f6a074de2c60f
Author: shuke <[email protected]>
AuthorDate: Fri Jul 10 10:03:03 2026 +0800

    [fix](regression) fix full compaction run status case (#65360)
    
    ## Summary
    
    - Fix the Groovy scope bug in `test_full_compaciton_run_status`.
    - Keep the full-compaction trigger response in loop-scoped `runCode`,
    `runOut`, and `runErr` variables so the `do ... while` condition can
    parse the latest response instead of resolving `out` as a missing
    `Suite` property.
    
    ## Testing
    
    - [x] `git diff --check --
    
regression-test/suites/fault_injection_p0/test_full_compaciton_run_status.groovy`
    - [x] `groovy -e "new GroovyShell().parse(new
    
File('/Users/keshu/dudu/fix-cases/worktrees/doris-full-compaction-run-status-branch41/regression-test/suites/fault_injection_p0/test_full_compaciton_run_status.groovy'))"`
    - [ ] Full regression case run, not run locally because it requires a
    live Doris regression environment with BE debug-point support.
---
 .../fault_injection_p0/test_full_compaciton_run_status.groovy      | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git 
a/regression-test/suites/fault_injection_p0/test_full_compaciton_run_status.groovy
 
b/regression-test/suites/fault_injection_p0/test_full_compaciton_run_status.groovy
index 1279530aad6..884f7f27bb0 100644
--- 
a/regression-test/suites/fault_injection_p0/test_full_compaciton_run_status.groovy
+++ 
b/regression-test/suites/fault_injection_p0/test_full_compaciton_run_status.groovy
@@ -66,12 +66,13 @@ suite("test_full_compaction_run_status","nonConcurrent") {
             backend_id = tablet.BackendId
 
             def times = 1
+            def runCode, runOut, runErr
             do{
-                def (code, out, err) = 
be_run_full_compaction(backendId_to_backendIP.get(backend_id), 
backendId_to_backendHttpPort.get(backend_id), tablet_id)
-                logger.info("Run compaction: code=" + code + ", out=" + out + 
", err=" + err)
+                (runCode, runOut, runErr) = 
be_run_full_compaction(backendId_to_backendIP.get(backend_id), 
backendId_to_backendHttpPort.get(backend_id), tablet_id)
+                logger.info("Run compaction: code=" + runCode + ", out=" + 
runOut + ", err=" + runErr)
                 ++times
                 sleep(1000)
-            } while (parseJson(out.trim()).status.toLowerCase()!="success" && 
times<=10)
+            } while (parseJson(runOut.trim()).status.toLowerCase()!="success" 
&& times<=10)
 
             def (code, out, err) = 
be_get_compaction_status(backendId_to_backendIP.get(backend_id), 
backendId_to_backendHttpPort.get(backend_id), tablet_id)
             logger.info("Get compaction status: code=" + code + ", out=" + out 
+ ", err=" + err)


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

Reply via email to