[ARROW-2641] [C++] Investigate spurious memset() calls

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.9.0
• Fix Version/s: 0.10.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/19036
• Assignee: Antoine Pitrou
• Reporter: Antoine Pitrou
• Created: 28/May/18 12:50
• Updated: 11/Jan/23 07:22
• Resolved: 31/May/18 15:38

Description

builder.cc has TODO statements of the form:

  // TODO(emkornfield) valgrind complains without this
  memset(data_->mutable_data(), 0, static_cast<size_t>(nbytes));

Ideally we shouldn't have to zero-initialize a data buffer before writing to it.

Issue Links

• supercedes: ARROW-2622 [C++] Array methods IsNull and IsValid are not complementary
• links to: GitHub Pull Request #2091