This is an automated email from the ASF dual-hosted git repository.
yangjie01 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new b14eb1e8d6cb [SPARK-52804][BUILD][FOLLOWUP] Revert Java minimum
version check for Maven
b14eb1e8d6cb is described below
commit b14eb1e8d6cbf0032684c8043b7713b578059eb1
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Tue Jul 22 10:09:09 2025 +0800
[SPARK-52804][BUILD][FOLLOWUP] Revert Java minimum version check for Maven
### What changes were proposed in this pull request?
In this pr, the Java minimum version check on the Maven side is temporarily
revert because it leads to incorrect judgments when `-Djava.version=17` is
specified:
Running `build/mvn clean install -DskipTests -Djava.version=17` results in
an error:
```
[ERROR] Failed to execute goal
org.apache.maven.plugins:maven-enforcer-plugin:3.6.0:enforce (enforce-versions)
on project spark-parent_2.13:
[ERROR] Rule 1: org.apache.maven.enforcer.rules.version.RequireJavaVersion
failed with message:
[ERROR] The Java version used to build the project is outdated. Please use
17.0.11 or later.
[ERROR] -> [Help 1]
```
This occurs because when `-Djava.version=17` is specified, both the
Detected Java Version and the Normalized Java Version are set to 17 instead of
the actual Java version.
### Why are the changes needed?
These changes are necessary to temporarily restore Maven daily tests.
Further fixes will be implemented once a reasonable solution is found. This
issue does not affect sbt.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
- Passed GitHub Actions
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #51598 from LuciferYang/SPARK-52804-FOLLOWUP.
Lead-authored-by: yangjie01 <yangji...@baidu.com>
Co-authored-by: YangJie <yangji...@baidu.com>
Signed-off-by: yangjie01 <yangji...@baidu.com>
---
pom.xml | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/pom.xml b/pom.xml
index 5aeb80f3f510..502642672995 100644
--- a/pom.xml
+++ b/pom.xml
@@ -2630,8 +2630,7 @@
<version>${maven.version}</version>
</requireMavenVersion>
<requireJavaVersion>
- <version>[${java.minimum.version},)</version>
- <message>The Java version used to build the project is
outdated. Please use ${java.minimum.version} or later.</message>
+ <version>${java.version}</version>
</requireJavaVersion>
<bannedDependencies>
<excludes>
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
