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]