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

    https://github.com/apache/spark/pull/21280#discussion_r187192271
  
    --- Diff: 
core/src/test/scala/org/apache/spark/scheduler/SparkListenerSuite.scala ---
    @@ -583,3 +587,23 @@ private class 
FirehoseListenerThatAcceptsSparkConf(conf: SparkConf) extends Spar
         case _ =>
       }
     }
    +
    +private class SlowDeserializable(var readDuration: Int)
    +  extends Externalizable {
    +
    +  def this() = this(0)
    --- End diff --
    
    Is this needed for deserialization? I thought it was ok to not have a 
default constructor.


---

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

Reply via email to