stevenzwu commented on code in PR #15630: URL: https://github.com/apache/iceberg/pull/15630#discussion_r3120694420
########## format/spec.md: ########## @@ -123,9 +128,22 @@ Tables do not require random-access writes. Once written, data and metadata file Tables do not require rename, except for tables that use atomic rename to implement the commit operation for new metadata files. +### Relative Locations in Metadata + +Version 4 of the Iceberg spec adds support for relative locations in metadata, enabling tables to be relocated without rewriting metadata files. Key changes include: + +* Support for relative locations in all metadata tracked path fields, resolved against the table's base location +* The table `location` field becomes optional, allowing the table location to be: + * Provided by an owning catalog + * Inferred from the metadata file location or storage layout + * Supplied directly where necessary Review Comment: I see. The first 2 bullet points are for read path on how to resolve the base location. This bullet is for the write path (like `register_table` or `create_table`). But the current wording `Supplied directly where necessary` seems a bit too abstract to get the right context. what about `Supplied directly when creating or registering a table`? -- 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]
