Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22855#discussion_r231213322
--- Diff:
core/src/test/scala/org/apache/spark/serializer/KryoSerializerSuite.scala ---
@@ -431,9 +434,11 @@ class KryoSerializerSuite extends SparkFunSuite with
SharedSparkContext {
ser.deserialize[HashMap[Int, List[String]]](serializedMap)
}
- private def testSerializerInstanceReuse(autoReset: Boolean,
referenceTracking: Boolean): Unit = {
+ private def testSerializerInstanceReuse(
+ autoReset: Boolean, referenceTracking: Boolean, usePool: Boolean):
Unit = {
--- End diff --
Total nit: there are a couple continuation indents that are 3 rather than 4
spaces. Don't bother, unless you need to make other updates.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
