[GitHub] [iceberg] kbendick commented on pull request #4951: [TEST] Core - Upgrade Parquet to 1.12.3 to get Zstd Buffer Pool by default

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

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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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