FANNG1 opened a new pull request, #10385:
URL: https://github.com/apache/gravitino/pull/10385

   ### What changes were proposed in this pull request?
   
   This PR removes the dedicated `release` Gradle task path and centralizes 
JDK8 compatibility logic in the root build configuration.
   
   1. Removed the root `release` task in `build.gradle.kts`.
   2. Updated `dev/release/release-build.sh` to use `build -x test` instead of 
`release`.
   3. Centralized JDK8 module matching logic in root `build.gradle.kts` with 
clearer project-name/prefix rules.
   4. Applied `compileJava` `--release 8` and test-side `--release 17` + 
`TargetJvmVersion=17` consistently for the targeted modules.
   5. Removed duplicated test JVM compatibility wiring from 
`clients/client-java/build.gradle.kts`.
   6. Replaced `Map.of()` with `Collections.emptyMap()` in Flink connector code 
to keep Java 8 API compatibility.
   
   ### Why are the changes needed?
   
   The previous flow depended on a separate `./gradlew release` path to produce 
JDK8-compatible artifacts, which was hard to maintain and easy to misuse.
   
   This change makes `./gradlew build` the single path and keeps JDK8 
compatibility checks/build behavior in one place, reducing duplicated logic and 
configuration drift.
   
   Fix: #10262
   
   ### Does this PR introduce _any_ user-facing change?
   
   No user-facing API or property-key changes.
   This is a build-system/internal compatibility workflow cleanup.
   
   ### How was this patch tested?
   
   Ran formatting and targeted compilation checks:
   
   1. `./gradlew spotlessApply`
   2. `./gradlew :spark-connector:spark-common:compileJava 
:spark-connector:spark-3.3:compileJava :spark-connector:spark-3.4:compileJava 
:spark-connector:spark-3.5:compileJava :flink-connector:flink:compileJava 
:flink-connector:flink-runtime:compileJava :clients:client-java:compileJava 
:clients:client-java-runtime:compileJava :bundles:aliyun-bundle:compileJava 
:bundles:aws-bundle:compileJava :bundles:azure-bundle:compileJava 
:bundles:gcp-bundle:compileJava`
   3. `./gradlew :spark-connector:spark-common:compileTestJava 
:spark-connector:spark-3.3:compileTestJava 
:spark-connector:spark-3.4:compileTestJava 
:spark-connector:spark-3.5:compileTestJava 
:flink-connector:flink:compileTestJava :clients:client-java:compileTestJava`
   4. Verified `release` task is removed from task list.
   


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

Reply via email to