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: [email protected]
For additional commands, e-mail: [email protected]
