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

    https://github.com/apache/spark/pull/19222#discussion_r138745656
  
    --- Diff: 
common/unsafe/src/main/java/org/apache/spark/unsafe/array/ByteArrayMethods.java 
---
    @@ -46,6 +47,42 @@ public static int roundNumberOfBytesToNearestWord(int 
numBytes) {
        * @return true if the arrays are equal, false otherwise
        */
       public static boolean arrayEquals(
    +      MemoryBlock leftBase, long leftOffset, MemoryBlock rightBase, long 
rightOffset, final long length) {
    +    int i = 0;
    +
    +    // check if stars align and we can get both offsets to be aligned
    --- End diff --
    
    lol :) If you scroll below, this is copy-paste from the L89 below .. so not 
introduced by this PR. I feel this might not be a typo and the author 
legitimately meant that.


---

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

Reply via email to