[GitHub] [incubator-iceberg] aokolnychyi commented on issue #63: Use Iceberg writers for Parquet data written from Spark.

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org