Github user kiszk commented on a diff in the pull request: https://github.com/apache/spark/pull/19222#discussion_r147573040 --- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/memory/HeapMemoryAllocator.java --- @@ -73,6 +73,12 @@ public MemoryBlock allocate(long size) throws OutOfMemoryError { @Override public void free(MemoryBlock memory) { + assert(memory instanceof ByteArrayMemoryBlock || memory instanceof IntArrayMemoryBlock || --- End diff -- Good catch, you are right.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org