fallintoplace opened a new pull request, #1220: URL: https://github.com/apache/arrow-go/pull/1220
## Summary - Stop `spacedExpand` as soon as the remaining decoded prefix is already in its final positions. - Avoid the final no-op `copy` for aligned valid prefixes. - Add focused coverage for empty, all-null, leading, middle, trailing, clustered, alternating, and offset bitmaps. - Add a 65,536-value benchmark for `int32` and `BYTE_ARRAY`. ## Benchmark - **Machine:** Apple M1 Pro, arm64 - **Go:** 1.26.3 - **Command:** `go test ./parquet/internal/encoding -run '^$' -bench '^BenchmarkSpacedExpand$' -benchmem -benchtime=50ms -count=3` - **Workloads:** trailing, late, clustered, random 10% nulls, and alternating validity layouts - **Result:** `0 B/op` and `0 allocs/op` for every case. CPU impact is small and layout-dependent, with the clearest benefit when the reverse walk reaches an already-aligned prefix. ## Tests - `go test ./parquet/internal/encoding` - `go test ./parquet/internal/... ./parquet/file ./parquet/metadata ./parquet/schema ./parquet/variant ./parquet/compress ./parquet/cmd/...` -- 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]
