laskoviymishka opened a new pull request, #1059: URL: https://github.com/apache/iceberg-go/pull/1059
Closes #1002. v3 spec requires `"current-snapshot-id": null` for empty tables, not the field omitted entirely (Java apache/iceberg#12335). omitempty on the *int64 field was dropping the key when nil; metadataV3 now has a MarshalJSON that re-injects the key with explicit null after the snapshots field — Java's declaration order — while preserving v1/v2 omitempty behaviour, since v1/v2 fixtures rely on the -1 sentinel. The injection walks the marshaled JSON via json.Decoder so the surrounding field order is preserved (a map round-trip would resort alphabetically and break byte-level diffs against Java-produced metadata.json). -- 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]
