kbendick commented on PR #4951: URL: https://github.com/apache/iceberg/pull/4951#issuecomment-1170298698
> @kbendick, what's the status of this? Should Iceberg set `parquet.compression.codec.zstd.bufferPool.enabled=true` for all Parquet files? That sounds like a good thing to me. Yes I believe we should set that configuration by default. Either by upgrading the patch version of parquet, or setting it ourselves. If we choose to set it ourselves and _not_ upgrade the parquet library, I'll do a quick pass over the PRs added between 1.12.2 and 1.12.3 to make sure it's not somehow unsafe to add. I think upgrading the parquet patch version would likely be better in the longer term. -- 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]
