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

    https://github.com/apache/spark/pull/19675#discussion_r149103482
  
    --- Diff: core/src/main/scala/org/apache/spark/util/ClosureCleaner.scala ---
    @@ -38,12 +38,13 @@ private[spark] object ClosureCleaner extends Logging {
         // Copy data over, before delegating to ClassReader - else we can run 
out of open file handles.
         val className = cls.getName.replaceFirst("^.*\\.", "") + ".class"
         val resourceStream = cls.getResourceAsStream(className)
    -    // todo: Fixme - continuing with earlier behavior ...
    -    if (resourceStream == null) return new ClassReader(resourceStream)
    --- End diff --
    
    Note that this already immediately generated an exception from `ClassReader`


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to