Re: [PR] REST: Set fresh IDs on create-table [iceberg-python]

2024-01-30 Thread via GitHub
rdblue commented on code in PR #327: URL: https://github.com/apache/iceberg-python/pull/327#discussion_r1471795199 ## pyiceberg/catalog/rest.py: ## @@ -447,13 +447,14 @@ def create_table( sort_order: SortOrder = UNSORTED_SORT_ORDER, properties: Properties = EMP

[PR] REST: Set fresh IDs on create-table [iceberg-python]

2024-01-30 Thread via GitHub
Fokko opened a new pull request, #327: URL: https://github.com/apache/iceberg-python/pull/327 For the other implementation we do this when writing the metadata, but for the REST catalog we just post the schema itself. The REST spec does not mention anything about setting fresh IDs,