Simplify StatsRewriteCtx rules by leveraging BoundExpressions - #9643
robert3005 wants to merge 4 commits into
2 benchmarks regressed
⚠️ Unknown Walltime execution environment detected
Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.
For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.
⚠️ Different runtime environments detected
Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.
⚡ 7 improved benchmarks
❌ 2 regressed benchmarks
✅ 2184 untouched benchmarks
⏩ 218 skipped benchmarks1
Warning
Please fix the performance issues or acknowledge them on CodSpeed.
Performance Changes
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | WallTime | arrow_checked_add_u32_neon[16384] |
12.2 µs | 20.4 µs | -39.95% |
| ❌ | WallTime | filtered_owned_i64_avx2[OneNullInEight] |
21.8 µs | 25.8 µs | -15.44% |
| ⚡ | WallTime | mul_u64_nonnull_neon |
20.6 µs | 15.4 µs | +33.6% |
| ⚡ | Simulation | decompress[u64, (4000, 1024)] |
86.1 µs | 70.5 µs | +22.05% |
| ⚡ | WallTime | mul_i64_nonnull_neon |
20.1 µs | 17.3 µs | +16.54% |
| ⚡ | WallTime | multiply_shapes_neon[(16384, PerRowPerRow)] |
20.1 µs | 17.3 µs | +16.17% |
| ⚡ | WallTime | dict_canonicalize_gt_u8_neon[1000000] |
559.8 µs | 487.1 µs | +14.93% |
| ⚡ | Simulation | allocate_drop_arrow[0] |
456.9 ns | 402.7 ns | +13.45% |
| ⚡ | WallTime | mul_u32_nonnull_avx512 |
6.2 µs | 5.6 µs | +11.36% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing rk/boundstatsrewrite (27867b3) with develop (e3b8eb2)
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. ↩