alamb commented on a change in pull request #1322:
URL: https://github.com/apache/arrow-rs/pull/1322#discussion_r809024299
##########
File path: parquet/src/file/metadata.rs
##########
@@ -467,6 +469,17 @@ impl ColumnChunkMetaData {
self.statistics.as_ref()
}
+ /// Returns `true` if this column chunk contains page encoding stats,
`false` otherwise.
+ pub fn has_page_encoding_stats(&self) -> bool {
Review comment:
Here is my proposal:
1. Merge this PR as is (it has some small clippy thing we need to fix to get
the CI clean)
2. Prepare a PR to remove all the has_XXX methods that have been added for
extended metadata so that we don't add them to a released version
3. Prepare a PR to remove the other (pre-existing has_XXX methods):
https://github.com/apache/arrow-rs/issues/1332
--
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]
