laskoviymishka opened a new pull request, #1056: URL: https://github.com/apache/iceberg-go/pull/1056
Closes #1049. ReadDV used to pass -1 to DeserializeDV unconditionally, skipping the spec-mandated cardinality check entirely. The puffin blob property "cardinality" — what the writer attaches — is now read out of the BlobMetadata that matches the manifest entry's (offset, size) and fed into DeserializeDV, so a truncated bitmap whose CRC still validates over the bytes that are present gets caught. Property missing → skip validation for backward-compat with writers that omit it (CRC still applies). Property present but unparseable → hard error; silently falling back would mask writer bugs. -- 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]
