fallintoplace opened a new pull request, #1529: URL: https://github.com/apache/iceberg-go/pull/1529
## Summary - convert timestamp-nanosecond values to microseconds before bucket hashing - use floor division so pre-epoch sub-microsecond values normalize correctly - keep Apply, Transformer, and projection behavior aligned ## Why The Iceberg spec requires timestamp-nanosecond bucketing to produce the same result as the equivalent microsecond timestamp. Hashing raw nanoseconds produced incompatible buckets and could also project predicates to the wrong partition. This matches the current Java Iceberg implementation. ## Testing - `go test -count=1 ./...` - `go run github.com/golangci/golangci-lint/v2/cmd/[email protected] run --timeout=10m` - regression coverage for positive, sub-microsecond, and pre-epoch values Fixes #1498 -- 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]
