wgtmac commented on issue #14870: URL: https://github.com/apache/arrow/issues/14870#issuecomment-1341084096
> AFAIK, the statistics in the data page header has been deprecated after column index (part of the page index) has been introduced. > > CMIW, parquet cpp is in good state with the parquet spec: > > * On the read path, the statistics in the ColumnChunkMetaData already consumes `min_value` & `max_value` from the thrift object if available: https://github.com/apache/arrow/blob/master/cpp/src/parquet/metadata.cc#L97 > * On the write path, we have written both `min_value` and `min` for backward-compatibility: https://github.com/apache/arrow/blob/master/cpp/src/parquet/thrift_internal.h#L292 The confusing thing is that both `Statistics` and `EncodedStatistics` use `min` and `max` for both old and new meanings. -- 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]
