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

    https://github.com/apache/spark/pull/19077#discussion_r144037771
  
    --- Diff: 
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/codegen/UnsafeArrayWriter.java
 ---
    @@ -57,7 +57,7 @@ public void initialize(BufferHolder holder, int 
numElements, int elementSize) {
     
         // Grows the global buffer ahead for header and fixed size data.
         int fixedPartInBytes =
    -      ByteArrayMethods.roundNumberOfBytesToNearestWord(elementSize * 
numElements);
    +      (int)ByteArrayMethods.roundNumberOfBytesToNearestWord(elementSize * 
numElements);
    --- End diff --
    
    nit: extra space after `(int) `, also please update the other similar 
changes.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to