fallintoplace opened a new pull request, #1739: URL: https://github.com/apache/iceberg-go/pull/1739
## What changed - use the writer content as the default content of the returned manifest file - reject data files in delete manifests and delete files in data manifests - reject a conflicting manifest file content override - keep V1 behavior unchanged because V1 does not encode file content ## Why A delete manifest writer currently writes `content=deletes` into the Avro metadata, but `ToManifestFile` defaults the returned manifest descriptor to data. The writer also accepts entries with the wrong file content. This can produce a manifest list entry that contradicts the manifest it references. ## Testing - added coverage for content inheritance and mismatches - `go test ./...` - `go vet ./...` -- 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]
