hello-stephen opened a new pull request, #65891:
URL: https://github.com/apache/doris/pull/65891

   ## What changed
   
   - make `run-regression-test.sh` select and validate JDK 17 before invoking 
Maven
   - prefer a valid `JAVA_HOME` or `JDK_17`, with Linux and macOS discovery 
fallbacks
   - build the regression framework and Java UDF case jar with the same JDK 17
   - compile `java-udf-src` with `maven.compiler.release=17`
   
   ## Why
   
   The regression build depended on the caller to prepare `JAVA_HOME`. This 
split ownership between CI and the Doris build script and could produce 
incomplete case artifacts when the build required JDK 17.
   
   Keeping Java selection in `run-regression-test.sh` makes local and CI builds 
follow the same contract and removes the Java 8 UDF bytecode target from active 
branches.
   
   ## Validation
   
   - `bash -n run-regression-test.sh`
   - `git diff --check`
   - ran `./run-regression-test.sh --clean` while the incoming `JAVA_HOME` 
pointed to JDK 22; the script selected JDK 17 and Maven reported Java 17
   - built `regression-test/java-udf-src` with JDK 17 successfully
   - verified `Echo$EchoInt` has class major version 61 and the expected UDF 
classes are present in the assembled jar
   


-- 
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