fallintoplace opened a new pull request, #1251: URL: https://github.com/apache/arrow-go/pull/1251
## Summary\n\n- Use typed memo insertion in the shared numeric hash path.\n- Keep the existing normalized uint8, uint16, uint32, and uint64 mappings.\n- Avoid the per-value interface call to MemoTable.GetOrInsert.\n- Add coverage for int32, int64, float32, float64, and both null modes.\n- Add a numeric dictionary_encode benchmark.\n\n## Benchmark\n\nApple M1 Pro. Compared with main at 6b039a76. The benchmark uses 65,535 values, 100 unique values, 500ms per sample, and 5 samples. The table shows median results.\n\n| type | main | branch | change | allocs/op |\n| --- | ---: | ---: | ---: | ---: |\n| int32 | 1.168 ms | 0.989 ms | 15.3% faster | 56 -> 56 |\n| int64 | 1.138 ms | 1.016 ms | 10.8% faster | 56 -> 56 |\n| float32 | 1.662 ms | 1.114 ms | 33.0% faster | 64,935 -> 56 |\n| float64 | 1.707 ms | 1.251 ms | 26.7% faster | 64,935 -> 56 |\n\nCommand:\n\n\n\n## Checks\n\n- ok github.com/apache/arrow-go/v18/arrow/compute (cached) ok github.com/apache/arrow-go/v18/arrow/compute/exec (cached) ok github.com/apache/arrow-go/v18/arrow/compute/exprs (cached) ok github.com/apache/arrow-go/v18/arrow/compute/internal/kernels (cached)\n- ok github.com/apache/arrow-go/v18/arrow/compute/internal/kernels 1.312s ok github.com/apache/arrow-go/v18/arrow/compute 1.732s\n- \n- \n -- 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]
