wgtmac commented on issue #14870:
URL: https://github.com/apache/arrow/issues/14870#issuecomment-1341081231

   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


-- 
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]

Reply via email to