Public
  1. Public

arrow

Public

Commits

Wes McKinneyKorn, Uwe
d130a0006fc
Wes McKinney authored and Korn, Uwe committed a096eb12ee0
ARROW-2821: [C++] Remove redundant memsets in BooleanBuilder

After the work in ARROW-2790, we were zeroing the memory in two places.

Even though there was some memory zeroing logic in AppendNull and AppendNulls, there are still some cases where the data buffer might have set bits when there is null data. While the Arrow columnar format does not guarantee that boolean values in null slots will be false, zeroing things make the array contents coming from this path deterministic.

Author: Wes McKinney <wesm+git@apache.org>

Closes #2236 from wesm/ARROW-2821 and squashes the following commits:

e7dcbf9e <Wes McKinney> Expand comment
c102ceb9 <Wes McKinney> Remove additional memsets