[SPARK-26137][CORE] Use Java system property "file.separator" instead…

… of hard coded "/" in DependencyUtils
apache · Nov 22, 2018 · 49ce5a8 · 49ce5a8
1 parent ab61ddb
commit 49ce5a8
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala b/core/src/main/scala/org/apache/spark/deploy/DependencyUtils.scala
@@ -19,6 +19,7 @@ package org.apache.spark.deploy
 
 import java.io.File
 import java.net.URI
+import java.util.regex.Pattern
 
 import org.apache.commons.lang3.StringUtils
 import org.apache.hadoop.conf.Configuration
@@ -61,11 +62,12 @@ private[deploy] object DependencyUtils extends Logging {
       hadoopConf: Configuration,
       secMgr: SecurityManager): String = {
     val targetDir = Utils.createTempDir()
+    val fileSeparator = Pattern.quote(System.getProperty("file.separator"))
     Option(jars)
       .map {
         resolveGlobPaths(_, hadoopConf)
           .split(",")
-          .filterNot(_.contains(userJar.split("/").last))
+          .filterNot(_.contains(userJar.split(fileSeparator).last))
           .mkString(",")
       }
       .filterNot(_ == "")