[ARROW-8127] [C++] [Parquet] Incorrect column chunk metadata for multipage batch writes

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.17.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/24335
• Assignee: TP Boudreau
• Reporter: TP Boudreau
• Created: 16/Mar/20 04:56
• Updated: 11/Jan/23 07:58
• Resolved: 18/Mar/20 17:12

Description

When writing to a buffered column writer using PLAIN encoding, if the size of the batch supplied for writing exceeds the page size for the writer, the resulting file has an incorrect data_page_offset set in its column chunk metadata. This causes an exception to be thrown when reading the file (file appears to be too short to the reader).

For example, the attached code, which attempts to write a batch of 262145 Int32's (= 1048576 + 4 bytes) using the default page size of 1048576 bytes (with buffered writer, PLAIN encoding), fails on reading, throwing the error: "Tried reading 1048678 bytes starting at position 1048633 from file but only got 333".

The error is caused by the second page write tripping the conditional here https://github.com/apache/arrow/blob/master/cpp/src/parquet/column_writer.cc#L302, in the serialized in-memory writer wrapped by the buffered writer.

The fix builds the metadata with offsets from the terminal sink rather than the in memory buffered sink. A PR is coming.

Issue Links

• links to: GitHub Pull Request #6637