Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19222#discussion_r177295079
  
    --- Diff: 
common/unsafe/src/main/java/org/apache/spark/unsafe/array/LongArray.java ---
    @@ -89,6 +85,6 @@ public void set(int index, long value) {
       public long get(int index) {
         assert index >= 0 : "index (" + index + ") should >= 0";
         assert index < length : "index (" + index + ") should < length (" + 
length + ")";
    -    return Platform.getLong(baseObj, baseOffset + index * WIDTH);
    +    return memory.getLong(memory.getBaseOffset() + index * WIDTH);
    --- End diff --
    
    ditto


---

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

Reply via email to