etseidl commented on PR #34054:
URL: https://github.com/apache/arrow/pull/34054#issuecomment-1449032119

   > > Apologies if I missed it but do you plan to add truncation functionality 
for large stat values?
   > 
   > Large stats truncation is already implemented by parquet-cpp: 
https://github.com/apache/arrow/blob/main/cpp/src/parquet/statistics.h#L145. So 
we are good here.
   
   Does the referenced code only apply to statistics in the  chunk metadata 
(and page headers)? Statistics in the page indexes can and should be truncated 
since there are no guarantees that those statistics are actually contained in 
the page. See  [second 
bullet](https://github.com/apache/parquet-format/blob/master/PageIndex.md#technical-approach)
 under "Some observations:".


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