ConeyLiu commented on code in PR #1177: URL: https://github.com/apache/parquet-mr/pull/1177#discussion_r1403526389
########## parquet-column/src/main/java/org/apache/parquet/internal/column/columnindex/ColumnIndex.java: ########## @@ -57,4 +57,16 @@ public interface ColumnIndex extends Visitor<PrimitiveIterator.OfInt> { */ public List<ByteBuffer> getMaxValues(); + /** + * @return the unmodifiable list of the repetition level histograms for each page concatenated together; used for + * converting to the related thrift object + */ + public List<Long> getRepetitionLevelHistogram(); Review Comment: The `public` is redundant, but I think you want to keep the code alignment. -- 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: dev-unsubscr...@parquet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org