danielcweeks commented on code in PR #15630:
URL: https://github.com/apache/iceberg/pull/15630#discussion_r3276784874


##########
format/spec.md:
##########
@@ -168,6 +184,48 @@ All columns must be written to data files even if they 
introduce redundancy with
 
 Writers are not allowed to commit files with a partition spec that contains a 
field with an unknown transform.
 
+### Paths in Metadata
+
+Path strings stored in Iceberg metadata location fields are classified as one 
of two types:

Review Comment:
   We don't want to do this (see other comments on this topic).  We don't want 
to go back and define things that weren't defined for prior versions since it 
could introduce additional requirements on older versions.  The prior spec only 
referred to "fully-qualified" and "URI with Scheme" for fields and we're not 
trying to rewrite those versions of the spec.



-- 
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]

Reply via email to