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

    https://github.com/apache/spark/pull/19135#discussion_r139309495
  
    --- Diff: 
core/src/main/scala/org/apache/spark/storage/memory/MemoryStore.scala ---
    @@ -190,11 +190,11 @@ private[spark] class MemoryStore(
         // Initial per-task memory to request for unrolling blocks (bytes).
         val initialMemoryThreshold = unrollMemoryThreshold
         // How often to check whether we need to request more memory
    -    val memoryCheckPeriod = 16
    +    val memoryCheckPeriod = 
conf.getLong("spark.storage.unrollMemoryCheckPeriod", 16)
    --- End diff --
    
    OK, moved. Pls take a took.


---

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

Reply via email to