stevenzwu commented on code in PR #15630:
URL: https://github.com/apache/iceberg/pull/15630#discussion_r3120210405
##########
format/spec.md:
##########
@@ -1637,6 +1686,30 @@ The binary single-value serialization can be used to
store the lower and upper b
## Appendix E: Format version changes
+### Version 4
+
+Relative path support is added in v4.
+
+Reading v3 metadata for v4:
+
+* All location fields are treated as absolute paths
+* Any location field without a uri scheme prefix must prepend a scheme
component consistent with v4
Review Comment:
But it contradict with earlier sentences in this spec change. We said before
V4 all file paths are absolute paths and absolute path is defined as including
URI scheme.
```
**Absolute path** -- A path string that includes a
[URI](https://datatracker.ietf.org/doc/html/rfc3986#section-3.1) scheme and can
be used directly.
...
Prior to v4, all path fields must contain absolute paths.
```
--
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]
