fallintoplace opened a new pull request, #1619: URL: https://github.com/apache/iceberg-go/pull/1619
## What changed Clear the sign bit before converting the UUID-derived snapshot ID to `int64`. The UUID folding logic is extracted into a small deterministic helper for regression coverage. ## Why Negating a negative value does not make `math.MinInt64` positive because signed negation overflows back to the same value. Clearing the sign bit guarantees that generated snapshot IDs are nonnegative for every input. ## Testing - Added deterministic coverage for an XOR result with only the sign bit set - Added coverage showing the remaining value bits are preserved - `go test ./table` -- 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]
