jonashaag opened a new pull request, #358:
URL: https://github.com/apache/iceberg-python/pull/358

   I had to change the `metadata_collector` code due to 
https://github.com/dask/dask/issues/7977.
   
   For the `<unset>` case (no compression specified) the tests currently pass 
locally but they shouldn't as we never set zstd as the default. Not sure what's 
going on?
   
   Should we default to gzip as is the default for Iceberg according to 
[this](https://iceberg.apache.org/docs/latest/configuration/) document, or 
should we use a more reasonable and modern default like zstd level 3?


-- 
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-unsubscr...@iceberg.apache.org

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


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

Reply via email to