[PARQUET-1840] [C++] DecodeSpaced copies more values then necessary

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: cpp-4.0.0
• Component/s: parquet-cpp
• Labels: pull-request-available
• Assignee: Micah Kornfield
• Reporter: Micah Kornfield
• Created: 13/Apr/20 03:24
• Updated: 14/Apr/20 09:52
• Resolved: 14/Apr/20 09:52

Description

• It potentially touches memory twice with memset
• The stopping condition should be when the indices are equal

Changing these could provide some marginal performance improvements depending on data shape.

Issue Links

• links to: GitHub Pull Request #6908