[GitHub] [iceberg] renshangtao commented on pull request #5158: Doc:Add parquet,orc,avro delete codec and level

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

renshangtao commented on PR #5158:
URL: https://github.com/apache/iceberg/pull/5158#issuecomment-1173271704

   @hililiwei Yes, you are right.The parquet and avro is the same as orc.
   ``
   String codecAsString = config.get(DELETE_PARQUET_COMPRESSION);
   ``
   ``
   CompressionCodecName codec = codecAsString != null ? toCodec(codecAsString) : dataContext.codec();
   ``
   
   @rdblue What is your opinion? Do we need to expose them.
   
   Can we modify it like this
   Property | Default | Description
   -- | -- | --
   write.delete.orc.compression-codec | data compression codec | ORC compression codec: zstd, lz4, lzo, zlib, snappy, none
   
   
   
   In the current document the write.delete format.default was exposed
   
   Property | Default | Description
   -- | -- | --
   write.format.default | parquet | Default file format for the table; parquet, avro, or orc
   write.delete.format.default | data file format | Default delete file format for the table; parquet, avro, or orc
   
   
   


-- 
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