Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r177306568
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/array/LongArray.java ---
@@ -69,8 +64,9 @@ public long size() {
* Fill this all with 0L.
*/
public void zeroOut() {
+ long baseOffset = memory.getBaseOffset();
for (long off = baseOffset; off < baseOffset + length * WIDTH; off +=
WIDTH) {
--- End diff --
Good catch. These three unchanges seems to lead to failures.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
