You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2025/01/22 17:30:20 UTC
(spark) branch branch-3.5 updated: Revert "[SPARK-50946][CORE][TESTS] Add version check for Java 17.0.14 to make `UtilsSuite` test pass"
This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch branch-3.5
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-3.5 by this push:
new 5abaeac78ef2 Revert "[SPARK-50946][CORE][TESTS] Add version check for Java 17.0.14 to make `UtilsSuite` test pass"
5abaeac78ef2 is described below
commit 5abaeac78ef2f58af48342b63f41fa0febf770fd
Author: Dongjoon Hyun <do...@apache.org>
AuthorDate: Wed Jan 22 09:29:56 2025 -0800
Revert "[SPARK-50946][CORE][TESTS] Add version check for Java 17.0.14 to make `UtilsSuite` test pass"
This reverts commit bd2b8e53be3d1713134c6154a66ae455953bfc95.
---
core/src/test/scala/org/apache/spark/util/UtilsSuite.scala | 5 -----
1 file changed, 5 deletions(-)
diff --git a/core/src/test/scala/org/apache/spark/util/UtilsSuite.scala b/core/src/test/scala/org/apache/spark/util/UtilsSuite.scala
index 2f46c4d67603..1a7bfc64c23c 100644
--- a/core/src/test/scala/org/apache/spark/util/UtilsSuite.scala
+++ b/core/src/test/scala/org/apache/spark/util/UtilsSuite.scala
@@ -525,15 +525,10 @@ class UtilsSuite extends SparkFunSuite with ResetSystemProperties {
// The following 3 scenarios are only for the method: createDirectory(File)
// 6. Symbolic link
- // JAVA_RUNTIME_VERSION is like "17.0.14+7-LTS"
- lazy val javaVersion = Runtime.Version.parse(SystemUtils.JAVA_RUNTIME_VERSION)
val scenario6 = java.nio.file.Files.createSymbolicLink(new File(testDir, "scenario6")
.toPath, scenario1.toPath).toFile
if (Utils.isJavaVersionAtLeast21) {
assert(Utils.createDirectory(scenario6))
- } else if (javaVersion.feature() == 17 && javaVersion.update() >= 14) {
- // SPARK-50946: Java 17.0.14 includes JDK-8294193, so scenario6 can succeed.
- assert(Utils.createDirectory(scenario6))
} else {
assert(!Utils.createDirectory(scenario6))
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org