laskoviymishka opened a new pull request, #1060: URL: https://github.com/apache/iceberg-go/pull/1060
Closes #1057. Puffin spec requires `deletion-vector-v1` blobs to carry both `cardinality` and `referenced-data-file` properties. AddBlob already enforces the snapshot-id/sequence-number invariants for DV blobs; extend the same branch to reject blobs missing either property. Closes the gap that prompted the reader-side slog.Warn fallback in #1049 — once writers can't omit the property, the warning path stays unreachable from in-tree writes and only fires on cross-impl reads of files written by third-party tools. Two existing test fixtures that wrote DV blobs without the cardinality property are updated: TestRoundTrip in puffin_test.go and writePuffinWithDVBlob in dv's deletion_vector_test.go. -- 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]
