danielcweeks commented on PR #8623:
URL: https://github.com/apache/iceberg/pull/8623#issuecomment-1741542649

   I'm a little hesitant to make this change to the spec because I feel the 
spec is pretty clear.  Also, it almost reads like the spec is putting an 
ambiguous requirement ("reasonable attempt(s)") on engine implementors.
   
   I can appreciate there's confusion around whether it's acceptable to assume 
that the field identifiers are unique, but currently the spec states that 
uniqueness is the intent, but not the guarantee.


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