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

    https://github.com/apache/spark/pull/22855#discussion_r230420266
  
    --- 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 --
    
    Okay, if I'm understanding you correctly I should set the default to true, 
and remove the documentation. I will go ahead and do that, if I misunderstood 
let me know. Thanks


---

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

Reply via email to