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

    https://github.com/apache/spark/pull/22855#discussion_r230421073
  
    --- Diff: 
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
    @@ -84,6 +85,7 @@ class KryoSerializer(conf: SparkConf)
       private val avroSchemas = conf.getAvroSchema
       // whether to use unsafe based IO for serialization
       private val useUnsafe = conf.getBoolean("spark.kryo.unsafe", false)
    +  private val usePool = conf.getBoolean("spark.kryo.pool", false)
    --- End diff --
    
    Yeah I think that's a fine position to take, if we can't think of a reason 
to disable it other than the theoretical unknown unknown bug out there.


---

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

Reply via email to