wgtmac commented on code in PR #14603:
URL: https://github.com/apache/arrow/pull/14603#discussion_r1037929261
##########
cpp/src/parquet/column_reader.cc:
##########
@@ -386,6 +441,12 @@ std::shared_ptr<Page> SerializedPageReader::NextPage() {
throw ParquetException("Invalid page header");
}
+ EncodedStatistics page_statistics;
Review Comment:
Renaming it to data_page_statistics? For now it is out of the scope of data
page, so we'd state clearly that it is meaningful for data page only.
--
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]
