Skip to content

Commit 3c8fe09

Browse files
authored
GH-45669: [C++][Parquet] Add missing ParquetFileReader::GetReadRanges() definition (#45684)
### Rationale for this change We only have a definition of it in an implementation private class but it's not defined in public `ParquetFileReader` class. ### What changes are included in this PR? Define `ParquetFileReader::GetReadRanges()` . ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * GitHub Issue: #45669 Authored-by: Sutou Kouhei <[email protected]> Signed-off-by: Sutou Kouhei <[email protected]>
1 parent 13c898c commit 3c8fe09

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

cpp/src/parquet/file_reader.cc

+10
Original file line numberDiff line numberDiff line change
@@ -923,6 +923,16 @@ void ParquetFileReader::PreBuffer(const std::vector<int>& row_groups,
923923
file->PreBuffer(row_groups, column_indices, ctx, options);
924924
}
925925

926+
::arrow::Result<std::vector<::arrow::io::ReadRange>> ParquetFileReader::GetReadRanges(
927+
const std::vector<int>& row_groups, const std::vector<int>& column_indices,
928+
int64_t hole_size_limit, int64_t range_size_limit) {
929+
// Access private methods here
930+
SerializedFile* file =
931+
::arrow::internal::checked_cast<SerializedFile*>(contents_.get());
932+
return file->GetReadRanges(row_groups, column_indices, hole_size_limit,
933+
range_size_limit);
934+
}
935+
926936
::arrow::Future<> ParquetFileReader::WhenBuffered(
927937
const std::vector<int>& row_groups, const std::vector<int>& column_indices) const {
928938
// Access private methods here

0 commit comments

Comments
 (0)