aokolnychyi commented on issue #63: Use Iceberg writers for Parquet data 
written from Spark.
URL: https://github.com/apache/incubator-iceberg/pull/63#issuecomment-474552196
 
 
   I think @prodeezy meant that the metrics are not actually fetched from the 
footer as of today. Correct me if I am wrong, but we still use 
`ParquetWriteAdapter` right now. It doesn't read the footer. 
   
   Anyway, I believe collecting min/max stats is a side-effect of switching to 
Iceberg writers. The main feature is that we actually switch to Iceberg 
writers, so it makes sense to keep the PR name as it is.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to