fatemehp commented on code in PR #14142:
URL: https://github.com/apache/arrow/pull/14142#discussion_r988440422
##########
cpp/src/parquet/column_reader.h:
##########
@@ -277,9 +277,15 @@ class RecordReader {
virtual ~RecordReader() = default;
/// \brief Attempt to read indicated number of records from column chunk
+ /// Note that for repeated fields, a record may have more than one value
+ /// and all of them are read.
/// \return number of records read
virtual int64_t ReadRecords(int64_t num_records) = 0;
+ /// \brief Attempt to skip indicated number of records from column chunk
+ /// \return number of records skipped
Review Comment:
Done.
##########
cpp/src/parquet/column_reader.h:
##########
@@ -164,7 +164,7 @@ class TypedColumnReader : public ColumnReader {
// may be less than the number of repetition and definition levels. With
// nested data this is almost certainly true.
//
- // Set def_levels or rep_levels to nullptr if you want to skip reading them.
+ // Set def_levels or rep_levels to nullptr if you want to reading them.
Review Comment:
Done.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
