fallintoplace opened a new pull request, #1885: URL: https://github.com/apache/iceberg-go/pull/1885
## Summary - Reuse the canonical partition tuple in `AddSurvivorWithSpec`. - Avoid sorting and formatting the same partition twice. - Keep empty-partition and negative-sequence behavior unchanged. ## Benchmark On Apple M1 Pro, 5 runs of `BenchmarkAddSurvivorWithSpec`: - 1 field: 72 -> 32 B/op, 7 -> 4 allocs/op - 8 fields: 624 -> 456 B/op, 28 -> 16 allocs/op - 32 fields: 2544 -> 1816 B/op, 80 -> 42 allocs/op The 32-field case was about 40% faster in the local comparison. ## Checks - `go test ./... -count=1` - `go test -race ./table/compaction -count=1` - `go vet ./table/compaction` -- 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]
