rdblue commented on code in PR #17822: URL: https://github.com/apache/iceberg/pull/17822#discussion_r4354603474
########## format/spec.md: ########## @@ -1878,10 +1991,12 @@ Hash results are not dependent on decimal scale, which is part of the type, not Schemas are serialized as a JSON object with the same fields as a struct in the table below, and the following additional fields: -| v1 | v2 and v3 |Field|JSON representation|Example| -| ---------- | ---------- |--- |--- |--- | -| _optional_ | _required_ |**`schema-id`**|`JSON int`|`0`| -| _optional_ | _optional_ |**`identifier-field-ids`**|`JSON list of ints`|`[1, 2]`| +| v1 | v2 and v3 | v4 |Field|JSON representation|Example| +| ---------- | ---------- | ---------- |--- |--- |--- | +| _optional_ | _required_ | _required_ |**`schema-id`**|`JSON int`|`0`| +| _optional_ | _optional_ | _removed_ |**`identifier-field-ids`**|`JSON list of ints`|`[1, 2]`| Review Comment: I think we leave a field blank when it is removed. -- 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]
