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

    https://github.com/apache/spark/pull/20706#discussion_r171879418
  
    --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
    @@ -1018,70 +1016,18 @@ private[spark] object Utils extends Logging {
         " " + (System.currentTimeMillis - startTimeMs) + " ms"
       }
     
    -  private def listFilesSafely(file: File): Seq[File] = {
    -    if (file.exists()) {
    -      val files = file.listFiles()
    -      if (files == null) {
    -        throw new IOException("Failed to list files for dir: " + file)
    -      }
    -      files
    -    } else {
    -      List()
    -    }
    -  }
    -
    -  /**
    -   * Lists files recursively.
    -   */
    -  def recursiveList(f: File): Array[File] = {
    --- End diff --
    
    Yes this is only used by test suites for now, but can you still keep this 
in `Utils` so we don't have to move it back when we want to use it in the 
production code (I assume this may happen someday). 


---

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

Reply via email to