[ARROW-6005] [C++] parquet::arrow::FileReader::GetRecordBatchReader() does not behave as documented since ARROW-1012

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.14.0, 0.14.1
• Fix Version/s: 0.15.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/22411
• Assignee: Hatem Helal
• Reporter: Martin
• Created: 22/Jul/19 21:08
• Updated: 11/Jan/23 07:44
• Resolved: 08/Aug/19 16:35

Description

GetRecordBatchReader() should

"Return a RecordBatchReader of row groups selected from row_group_indices, the
ordering in row_group_indices matters." (that is what the doxygen string says),

but:

Since change ARROW-1012, it ignores the row_group_indices argument.

The row_group_indices_ in the RowGroupRecordBatchReader that is created are never used.

Either the documentation should be changed, or the behavior should be reverted. I would prefer the latter, as I do not know how to make sure to read a specific row groups anymore...

Issue Links

• links to: GitHub Pull Request #5012