Github user kiszk commented on the issue:

    https://github.com/apache/spark/pull/19077
  
    @cloud-fan IIUC, this PR does not change alignment policy for page 
allocation. Alignment policy still depends on JVM. This PR changes reuse policy 
in memory pool.
    
    If you are asking about round policy (i.e. 8-byte in this PR) for memory 
reuse, I do not have strong opinion. 8-byte is also fine.
    If you are asking about allocation policy, it would be good to use 8-byte 
boundaries for an allocation as HotSpot and IBM JVMs do.


---

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

Reply via email to