Fix deprecation warning in btrblocks varbin scheme - #9819
Conversation
Signed-off-by: Robert Kruszewski <github@robertk.io>
Merging this PR will improve performance by 34.56%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | WallTime | arrow_checked_add_u32_neon[16384] |
20.4 µs | 12.3 µs | +66.3% |
| ⚡ | WallTime | arrow_checked_add_u32_avx2[16384] |
21.4 µs | 17.7 µs | +21.23% |
| ⚡ | WallTime | arrow_checked_add_u32_avx512[16384] |
21.2 µs | 17.6 µs | +20.84% |
Tip
Curious why performance improved? Comment @codspeedbot explain why performance improved on this PR, or directly use the CodSpeed MCP with your agent.
Comparing rk/fmt (f9884c4) with develop (2a5b1a7)
Footnotes
-
218 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
…take-neon Picks up #9819, which fixes the `VarBinBuilder::with_capacity` deprecation in `vortex-btrblocks` that was failing both `Rust (lint)` jobs on this PR. Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
This was a merge conflict