adamreeve commented on code in PR #45186:
URL: https://github.com/apache/arrow/pull/45186#discussion_r1922842225


##########
cpp/src/parquet/column_reader.cc:
##########
@@ -1603,7 +1603,12 @@ class TypedRecordReader : public 
TypedColumnReaderImpl<DType>,
     // another record start or exhausting the ColumnChunk
     int64_t level = levels_position_;
     if (at_record_start_) {
-      ARROW_DCHECK_EQ(0, rep_levels[levels_position_]);
+      if (rep_levels[levels_position_] != 0) {

Review Comment:
   Hmm yeah I guess there's the possibility there could be non-zero repetition 
levels when the maximum is supposed to be 0. I'm not sure it's worth reading 
the repetition levels just to check for that though?



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to