[GitHub] [iceberg] rdblue commented on a change in pull request #3892: Docs: Add enumeration supported by `write.parquet.compression-codec`

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

rdblue commented on a change in pull request #3892:
URL: https://github.com/apache/iceberg/pull/3892#discussion_r786307688



##########
File path: site/docs/configuration.md
##########
@@ -38,7 +38,7 @@ Iceberg tables support table properties to configure table behavior, like the de
 | write.parquet.row-group-size-bytes | 134217728 (128 MB) | Parquet row group size                             |
 | write.parquet.page-size-bytes      | 1048576 (1 MB)     | Parquet page size                                  |
 | write.parquet.dict-size-bytes      | 2097152 (2 MB)     | Parquet dictionary page size                       |
-| write.parquet.compression-codec    | gzip               | Parquet compression codec                          |
+| write.parquet.compression-codec    | gzip               | Parquet compression codec; uncompressed, snappy, gzip, lzo, brotli, lz4, zstd. Note that `zstd` requires `ZStandardCodec` to be installed before Hadoop 2.9.0, `brotli` requires `BrotliCodec` to be installed.                        |

Review comment:
       Maybe this should be a link to Parquet docs instead?
   
   This doesn't fit in a table, so we will need to move it or shorten it. I think making it a link fixes both problems.




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