Github user xuanyuanking commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21533#discussion_r195133036
  
    --- Diff: core/src/test/scala/org/apache/spark/SparkContextSuite.scala ---
    @@ -116,49 +116,52 @@ class SparkContextSuite extends SparkFunSuite with 
LocalSparkContext with Eventu
       test("basic case for addFile and listFiles") {
         val dir = Utils.createTempDir()
     
    +    // file and absolute path for normal path
         val file1 = File.createTempFile("someprefix1", "somesuffix1", dir)
         val absolutePath1 = file1.getAbsolutePath
     
    +    // file and absolute path for relative path
         val file2 = File.createTempFile("someprefix2", "somesuffix2", dir)
         val relativePath = file2.getParent + "/../" + 
file2.getParentFile.getName + "/" + file2.getName
         val absolutePath2 = file2.getAbsolutePath
     
    +    // file and absolute path for path with local scheme
    +    val file3 = File.createTempFile("someprefix3", "somesuffix3", dir)
    +    val localPath = "local://" + file3.getParent + "/../" + 
file3.getParentFile.getName +
    --- End diff --
    
    Got it, thanks.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to