Remove the unstable encodings feature - #9710
Conversation
Merging this PR will regress 1 benchmark
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | random_i8[0.5] |
67 µs | 90.5 µs | -25.97% |
| ⚡ | WallTime | arrow_checked_add_u32_neon[16384] |
20.4 µs | 12.8 µs | +59.12% |
| ⚡ | Simulation | random_i16[0.8] |
92.7 µs | 74.1 µs | +24.97% |
| ⚡ | Simulation | decompress[u64, (4000, 1024)] |
85.5 µs | 70.4 µs | +21.45% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing rk/remove-unstable-encodings (cbc1339) with develop (3170657)
Footnotes
-
164 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. ↩
-
1 benchmark was run, but is now archived. If it was deleted in another branch, consider rebasing to remove it from the report. Instead if it was added back, click here to restore it. ↩
2649c0b to
7102677
Compare
82cdf1c to
9db048d
Compare
08bcf78 to
b599fdd
Compare
Use enabled editions to filter the default writer and golden compressor configurations. Signed-off-by: Robert Kruszewski <github@robertk.io>
Signed-off-by: Robert Kruszewski <github@robertk.io>
Signed-off-by: Robert Kruszewski <github@robertk.io>
cbc1339 to
0af6657
Compare
unstable_encodings was only ever necessary to let us try things out without immediately introducing stability guarantees. We have editions now, that's the way things should be restricted, not via compile flags