morningman opened a new pull request, #65796:
URL: https://github.com/apache/doris/pull/65796

   ### What problem does this PR solve?
   
   Issue Number: close #xxx
   
   Related PR: #58914
   
   Problem Summary:
   
   The `BE UT (macOS)` job fails at the JDK version check, e.g. in
   https://github.com/apache/doris/actions/runs/29692786375/job/88208347509:
   
   ```
   Check JAVA version
   ERROR: The JAVA version is 25, it must be JDK-17.
   ```
   
   Root cause: #58914 migrated the runner from the deprecated `macos-13`
   (Intel / x64) to `macos-15`, which is Apple Silicon (arm64). GitHub's runner
   images expose the JDK path as `JAVA_HOME_<version>_<arch>`, so on arm64 it is
   `JAVA_HOME_17_arm64`, and the old `JAVA_HOME_17_X64` is unset. The workflow
   still read `JAVA_HOME_17_X64`, so `JAVA_HOME` became empty; the build then 
fell
   back to the system default Java (25) and `check_jdk_version` failed.
   
   Fix: read `JAVA_HOME_17_arm64` with a fallback to `JAVA_HOME_17_X64`, keeping
   the job working on Apple Silicon runners while staying compatible with Intel
   ones. The trailing-slash stripping of the original is preserved.
   
   Note: this workflow's paths-filter only triggers on `be/**` / `gensrc/**`
   changes, so this PR (which only edits the workflow) will not run the macOS
   BE UT job itself. The fix will be exercised by the next scheduled run or the
   next PR touching BE code. The env-var resolution was verified locally for 
both
   arm64 and x64 cases.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test
       - [x] No need to test or manual test. Explain why:
           - [x] Other reason: CI-workflow-only change; correctness verified 
locally by simulating the `JAVA_HOME_17_arm64` / `JAVA_HOME_17_X64` resolution 
on both architectures.
   
   - Behavior changed:
       - [x] No.
   
   - Does this need documentation?
       - [x] No.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to