smaheshwar-pltr commented on code in PR #1537:
URL: https://github.com/apache/iceberg-python/pull/1537#discussion_r1921652324
##########
mkdocs/docs/configuration.md:
##########
@@ -54,15 +54,18 @@ Iceberg tables support table properties to configure table
behavior.
### Write options
-| Key | Options |
Default | Description
|
-| -------------------------------------- | --------------------------------- |
------- |
-------------------------------------------------------------------------------------------
|
-| `write.parquet.compression-codec` | `{uncompressed,zstd,gzip,snappy}` |
zstd | Sets the Parquet compression coddec.
|
-| `write.parquet.compression-level` | Integer |
null | Parquet compression level for the codec. If not set, it is up to
PyIceberg |
-| `write.parquet.row-group-limit` | Number of rows |
1048576 | The upper bound of the number of entries within a single row group
|
-| `write.parquet.page-size-bytes` | Size in bytes |
1MB | Set a target threshold for the approximate encoded size of data pages
within a column chunk |
-| `write.parquet.page-row-limit` | Number of rows |
20000 | Set a target threshold for the maximum number of rows within a column
chunk |
-| `write.parquet.dict-size-bytes` | Size in bytes |
2MB | Set the dictionary page size limit per row group
|
-| `write.metadata.previous-versions-max` | Integer |
100 | The max number of previous version metadata files to keep before
deleting after commit. |
+| Key | Options
| Default | Description
|
+|------------------------------------------|-----------------------------------|---------|-------------------------------------------------------------------------------------------------------------------------------------|
+| `write.parquet.compression-codec` | `{uncompressed,zstd,gzip,snappy}`
| zstd | Sets the Parquet compression coddec.
|
+| `write.parquet.compression-level` | Integer
| null | Parquet compression level for the codec. If not set, it is up to
PyIceberg |
+| `write.parquet.row-group-limit` | Number of rows
| 1048576 | The upper bound of the number of entries within a single row group
|
+| `write.parquet.page-size-bytes` | Size in bytes
| 1MB | Set a target threshold for the approximate encoded size of data
pages within a column chunk |
+| `write.parquet.page-row-limit` | Number of rows
| 20000 | Set a target threshold for the maximum number of rows within a
column chunk |
+| `write.parquet.dict-size-bytes` | Size in bytes
| 2MB | Set the dictionary page size limit per row group
|
+| `write.metadata.previous-versions-max` | Integer
| 100 | The max number of previous version metadata files to keep before
deleting after commit. |
+| `write.object-storage.enabled` | Boolean
| True | Enables the
[ObjectStoreLocationProvider](configuration.md#objectstorelocationprovider)
that adds a hash component to file paths |
+| `write.object-storage.partitioned-paths` | Boolean
| True | Controls whether [partition values are included in file
paths](configuration.md#partition-exclusion) when object storage is enabled |
+| `write.py-location-provider.impl` | String of form `module.ClassName`
| null | Optional, [custom
LocationProvider](configuration.md#loading-a-custom-locationprovider)
implementation |
Review Comment:
(The above screenshot also shows how code/backticks hyperlinks look, I think
they're fine. This is now relevant because of
https://github.com/apache/iceberg-python/pull/1537#discussion_r1921652049.
--
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]
