Github user kiszk commented on the issue:

    https://github.com/apache/spark/pull/20636
  
    When I check callers of `BufferHolder.grow()`, some of them call 
`ByteArrayMethods.roundNumberOfBytesToNearestWord()` and other do not call it 
(i.e. implicitly ensure word-aligned).
    
    Is it better way to call 
`ByteArrayMethods.roundNumberOfBytesToNearestWord()` at `BufferHolder.grow()` 
instread of a caller to gurantee word-aligned?
    Then, we can check whether `UnsafeRow.getSizeInBytes()` is a multiple of 8 
in `BufferHolderSparkSubmitSuite`.


---

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

Reply via email to