laskoviymishka opened a new issue, #1004: URL: https://github.com/apache/iceberg-go/issues/1004
Parent: #589 The spec mandates `next-row-id` monotonically increases across snapshots. `MetadataBuilder.validateAndUpdateRowLineage` does an implicit guard via "snapshot's first-row-id must not be behind the table's current cursor," but does not explicitly assert that the cursor *advances* after applying the snapshot's `AddedRows`. A buggy producer setting `AddedRows` to 0 (or negative — the field is `*int64`) would silently leave the cursor unchanged or move it backwards. Add an explicit post-update check: reject if the new `nextRowID` is less than or equal to the previous one. Error message names both values for triage parity with #940. Tests: happy-path normal increment passes; a snapshot with `AddedRows = 0` is rejected; a snapshot with negative `AddedRows` is rejected. -- 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]
