Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19868#discussion_r181607863
  
    --- Diff: core/src/main/scala/org/apache/spark/rdd/HadoopRDD.scala ---
    @@ -279,6 +293,10 @@ class HadoopRDD[K, V](
               case e: IOException if ignoreCorruptFiles =>
                 logWarning(s"Skipped the rest content in the corrupted file: 
${split.inputSplit}", e)
                 finished = true
    +          case e: FileNotFoundException if ignoreMissingFiles =>
    +            logWarning(s"Skipped missing file: ${split.inputSplit}", e)
    +            finished = true
    +            null
    --- End diff --
    
    why returning null here?


---

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

Reply via email to