sungwy opened a new pull request, #17137: URL: https://github.com/apache/iceberg/pull/17137
The REST catalog spec does not explicitly state how a REST Catalog Server should handle field IDs on schema updates. This PR clarifies the docs to state: 1. Field IDs SHOULD NOT be reassigned on `AddSchemaUpdate`, as doing so would make the schema inconsistent with the field IDs already referenced by committed or co-committed data files. 2. Field IDs MAY be reassigned on `CreateTableRequest`, since a new table has no committed data referencing them yet. This matches the behavior implemented in Iceberg core, which the reference REST server (RCK / iceberg-rest-fixture) is built on, and which open source implementations such as Apache Polaris reuse. -- 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]
