[PARQUET-2423] Avoid allocating buffer obeject in RecordReader's SkipRecords

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: cpp-16.0.0
• Component/s: parquet-cpp
• Labels: pull-request-available
• Assignee: Jinpeng Zhou
• Reporter: Jinpeng Zhou
• Created: 27/Jan/24 03:07
• Updated: 23/Jun/24 03:33
• Resolved: 29/Feb/24 16:07

Description

Currently each invocation of SkipRecords() for non-repeated fields will create a brand new buffer object[1]. I think it probably worth keep the buffer object alive and just resize it for each skip, as the buffer is just a bitmap for records.

[1]https://github.com/apache/arrow/blob/main/cpp/src/parquet/column_reader.cc#L1482

Issue Links

• links to: GitHub Pull Request #39818