From a0cb17450ce90bd6fe8684563c891fc6bb6c4c43 Mon Sep 17 00:00:00 2001 From: FANNG Date: Fri, 6 Sep 2024 00:28:20 +0800 Subject: [PATCH] [#4758] correct Iceberg document location is immutable not reserved (#4808) ### What changes were proposed in this pull request? correct Iceberg document location is immutable not reserved ### Why are the changes needed? Fix: #4758 ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? document --- docs/lakehouse-iceberg-catalog.md | 25 +++++++++++++++---------- 1 file changed, 15 insertions(+), 10 deletions(-) diff --git a/docs/lakehouse-iceberg-catalog.md b/docs/lakehouse-iceberg-catalog.md index 738f0e432bb..0fc3af157c0 100644 --- a/docs/lakehouse-iceberg-catalog.md +++ b/docs/lakehouse-iceberg-catalog.md @@ -284,16 +284,21 @@ You can pass [Iceberg table properties](https://iceberg.apache.org/docs/1.5.2/co The Gravitino server doesn't allow passing the following reserved fields. -| Configuration item | Description | -|---------------------------------|---------------------------------------------------------| -| `comment` | The table comment. | -| `creator` | The table creator. | -| `location` | Iceberg location for table storage. | -| `current-snapshot-id` | The snapshot represents the current state of the table. | -| `cherry-pick-snapshot-id` | Selecting a specific snapshot in a merge operation. | -| `sort-order` | Selecting a specific snapshot in a merge operation. | -| `identifier-fields` | The identifier fields for defining the table. | -| `write.distribution-mode` | Defines distribution of write data | +| Configuration item | Description | Since Version | +|---------------------------|--------------------------------------------------------------------------------------|---------------| +| `comment` | The table comment, please use `comment` field in table meta instead. | 0.2.0 | +| `creator` | The table creator. | 0.2.0 | +| `current-snapshot-id` | The snapshot represents the current state of the table. | 0.2.0 | +| `cherry-pick-snapshot-id` | Selecting a specific snapshot in a merge operation. | 0.2.0 | +| `sort-order` | Iceberg table sort order, please use `SortOrder` in table meta instead. | 0.2.0 | +| `identifier-fields` | The identifier fields for defining the table. | 0.2.0 | +| `write.distribution-mode` | Defines distribution of write data, please use `distribution` in table meta instead. | 0.2.0 | + +Gravitino server doesn't allow to change such properties: + +| Configuration item | Description | Default value | Required | Since Version | +|--------------------|----------------------------------------------|---------------|----------|---------------| +| `location` | Iceberg location for table storage. | None | No | 0.2.0 | ### Table indexes