dbtsai commented on PR #8158: URL: https://github.com/apache/iceberg/pull/8158#issuecomment-1660890398
To avoid surprise to our users, I agree with the above comments. We should address the following two items as followup PRs. 1. Persistent the default value of compression codec in `write.parquet.compression-codec` for new table. 2. For existing table that doesn't set `write.parquet.compression-codec`, we should default to `gzip`, and set it. @stevenzwu I believe most of the modern runtime supports zstd parquet. Trino switched to use zstd parquet as default for Iceberg table for almost 2 years, and we never hear any compatibility issue. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
