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 3347496caae branch-4.1: [fix](test) skip test_flight_record when the 
frontend is not on the regression runner #65933 (#65946)
3347496caae is described below

commit 3347496caae16d729b1379a1c29c69400460312d
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Jul 24 11:05:44 2026 +0800

    branch-4.1: [fix](test) skip test_flight_record when the frontend is not on 
the regression runner #65933 (#65946)
    
    Cherry-picked from #65933
    
    Co-authored-by: 924060929 <[email protected]>
---
 .../suites/demo_p0/test_flight_record.groovy          | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/regression-test/suites/demo_p0/test_flight_record.groovy 
b/regression-test/suites/demo_p0/test_flight_record.groovy
index 652faf31305..a69f820a203 100644
--- a/regression-test/suites/demo_p0/test_flight_record.groovy
+++ b/regression-test/suites/demo_p0/test_flight_record.groovy
@@ -27,12 +27,29 @@ suite("test_flight_record", "nonConcurrent") {
         return
     }
 
+    // flightRecord starts the java flight recorder by the local jps/jcmd, so 
it only works when the
+    // frontend runs on the same machine as the regression runner, e.g. a 
local development cluster.
+    // Skip this demo when the frontend is not here, otherwise flightRecord 
will fail the suite
+    String feProcessName = "DorisFE"
+    boolean feOnThisMachine = false
+    try {
+        feOnThisMachine = "jps".execute().text.readLines().any { 
it.contains(feProcessName) }
+    } catch (Throwable t) {
+        // jps is shipped with the jdk, it doesn't exist if this machine only 
installs a jre
+        logger.info("Can not execute jps: ${t.getMessage()}")
+    }
+    if (!feOnThisMachine) {
+        logger.info("Process ${feProcessName} is not running on this machine, 
this case requires the "
+                + "frontend and the regression runner deployed together, skip 
test")
+        return
+    }
+
     flightRecord {
         // whether delete jfr file after callback, default is true
         cleanUp true
 
         // the process name, default is DorisFE
-        processName "DorisFE"
+        processName feProcessName
 
         // the jcmd extra config, default is empty
         extraConfig(["jdk.ObjectAllocationSample#throttle=\"100 /ns\""])


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

Reply via email to