wgtmac commented on code in PR #14556: URL: https://github.com/apache/arrow/pull/14556#discussion_r1014842198
########## cpp/src/parquet/reader_test.cc: ########## @@ -547,12 +547,12 @@ Column 1: b (INT32) --- Rows: 3 --- Column 0 Values: 18 Statistics Not Set - Compression: SNAPPY, Encodings: RLE PLAIN_DICTIONARY + Compression: SNAPPY, Encodings: PLAIN_DICTIONARY(DICT_PAGE) PLAIN_DICTIONARY Review Comment: The RLE encoding was used for def/rep levels. After this change, it only prints encoding of data. This is in sync with what parquet-mr does: https://github.com/apache/parquet-mr/blob/c00479d538353348726cf78da835078024161e55/parquet-cli/src/main/java/org/apache/parquet/cli/commands/ParquetMetadataCommand.java#L163 -- 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
