stevenzwu commented on code in PR #17198:
URL: https://github.com/apache/iceberg/pull/17198#discussion_r3589477214
##########
format/spec.md:
##########
@@ -674,6 +674,10 @@ A manifest file must store the partition spec and other
metadata as properties i
| _optional_ | _required_ | `format-version` | Table format version
number of the manifest as a string
|
| | _required_ | `content` | Type of content files
tracked by the manifest: "data" or "deletes"
|
+#### Content file uniqueness
+
+Within a snapshot, each content file may be referenced by at most one manifest
entry across all manifests. A snapshot in which more than one entry references
the same content file has undefined behavior. Writers are not required to
validate uniqueness at commit time.
+
Review Comment:
this seems like the right placement for the `Content file uniqueness`
section`.
but we probably should move line 681 to the `Manifest Entry Fields` section
below. it can probably be merged with line 685.
--
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]
