Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169983548
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/HeapMemoryAllocator.java
---
@@ -70,7 +71,7 @@ public MemoryBlock allocate(long size) throws
OutOfMemoryError {
}
}
long[] array = new long[numWords];
- MemoryBlock memory = new MemoryBlock(array,
Platform.LONG_ARRAY_OFFSET, size);
+ MemoryBlock memory = new OnHeapMemoryBlock(array,
Platform.LONG_ARRAY_OFFSET, size);
--- End diff --
ditto
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
