Optimize dependencies using cargo-shear#8715
Conversation
Merging this PR will improve performance by 10.62%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | slice_empty_vortex |
339.4 ns | 397.8 ns | -14.66% |
| ❌ | Simulation | encode_varbin[(1000, 32)] |
147.6 µs | 165 µs | -10.52% |
| ❌ | Simulation | compare_int_constant |
268.5 µs | 298.7 µs | -10.12% |
| ⚡ | Simulation | chunked_bool_canonical_into[(1000, 10)] |
26.5 µs | 16.2 µs | +63.29% |
| ⚡ | Simulation | bitwise_not_vortex_buffer_mut[128] |
273.6 ns | 215.3 ns | +27.1% |
| ⚡ | Simulation | bitwise_not_vortex_buffer_mut[1024] |
333.9 ns | 275.6 ns | +21.17% |
| ⚡ | Simulation | bitwise_not_vortex_buffer_mut[2048] |
427.8 ns | 369.4 ns | +15.79% |
| ⚡ | Simulation | chunked_varbinview_into_canonical[(100, 100)] |
306.9 µs | 272.2 µs | +12.75% |
| ⚡ | Simulation | true_count_vortex_buffer[128] |
638.3 ns | 580 ns | +10.06% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing adamg/cargo-shear-run-2026-07-10 (29f83f4) with develop (357a4ba)
Footnotes
-
4 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. ↩
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
db591e9 to
29f83f4
Compare
connortsui20
left a comment
There was a problem hiding this comment.
seems fine to me, triggered benchmarks just in case features are weird
Polar Signals Profiling ResultsLatest Run
Powered by Polar Signals Cloud |
Benchmarks: PolarSignals ProfilingVortex (geomean): 1.033x ➖ How to read Verdict and Engines
datafusion / vortex-file-compressed (1.033x ➖, 0↑ 2↓)
No file size changes detected. |
Benchmarks: TPC-H SF=1 on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.979x ➖, 0↑ 0↓)
datafusion / vortex-compact (0.975x ➖, 0↑ 0↓)
datafusion / parquet (0.976x ➖, 0↑ 1↓)
datafusion / arrow (0.983x ➖, 1↑ 0↓)
duckdb / vortex-file-compressed (0.986x ➖, 0↑ 0↓)
duckdb / vortex-compact (0.980x ➖, 0↑ 0↓)
duckdb / parquet (0.993x ➖, 0↑ 2↓)
duckdb / duckdb (0.990x ➖, 0↑ 0↓)
File Size Changes (10 files changed, -0.0% overall, 5↑ 5↓)
Totals:
|
Benchmarks: FineWeb NVMeVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.870x ✅, 7↑ 0↓)
datafusion / vortex-compact (0.905x ➖, 6↑ 0↓)
datafusion / parquet (0.886x ✅, 7↑ 0↓)
duckdb / vortex-file-compressed (0.891x ✅, 5↑ 0↓)
duckdb / vortex-compact (0.923x ➖, 2↑ 0↓)
duckdb / parquet (0.883x ✅, 7↑ 0↓)
File Size Changes (1 files changed, -0.0% overall, 0↑ 1↓)
Totals:
|
Benchmarks: Random AccessVortex (geomean): 0.917x ➖ How to read Verdict and Engines
unknown / unknown (0.952x ➖, 5↑ 0↓)
|
Benchmarks: TPC-DS SF=1 on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.031x ➖, 0↑ 3↓)
datafusion / vortex-compact (1.035x ➖, 0↑ 1↓)
datafusion / parquet (1.040x ➖, 2↑ 7↓)
duckdb / vortex-file-compressed (1.053x ➖, 0↑ 16↓)
duckdb / vortex-compact (1.009x ➖, 3↑ 2↓)
duckdb / parquet (1.013x ➖, 0↑ 2↓)
duckdb / duckdb (0.991x ➖, 3↑ 2↓)
File Size Changes (5 files changed, -0.1% overall, 1↑ 4↓)
Totals:
|
Benchmarks: Statistical and Population GeneticsVerdict: No clear signal (low confidence) How to read Verdict and Engines
duckdb / vortex-file-compressed (1.017x ➖, 0↑ 0↓)
duckdb / vortex-compact (1.010x ➖, 0↑ 0↓)
duckdb / parquet (1.003x ➖, 0↑ 0↓)
File Size Changes (1 files changed, -0.0% overall, 0↑ 1↓)
Totals:
|
Benchmarks: FineWeb S3Verdict: No clear signal (environment too noisy confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.165x ➖, 0↑ 2↓)
datafusion / vortex-compact (0.984x ➖, 1↑ 1↓)
datafusion / parquet (1.128x ➖, 0↑ 0↓)
duckdb / vortex-file-compressed (1.154x ➖, 0↑ 0↓)
duckdb / vortex-compact (1.147x ➖, 0↑ 1↓)
duckdb / parquet (1.081x ➖, 0↑ 0↓)
|
Benchmarks: Clickbench Sorted on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.028x ➖, 2↑ 2↓)
datafusion / parquet (1.068x ➖, 0↑ 3↓)
duckdb / vortex-file-compressed (1.086x ➖, 0↑ 4↓)
duckdb / parquet (1.006x ➖, 1↑ 0↓)
duckdb / duckdb (1.029x ➖, 0↑ 0↓)
File Size Changes (201 files changed, -0.0% overall, 99↑ 102↓)
Totals:
|
Benchmarks: TPC-H SF=10 on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.007x ➖, 3↑ 0↓)
datafusion / vortex-compact (1.012x ➖, 2↑ 0↓)
datafusion / parquet (1.017x ➖, 1↑ 0↓)
datafusion / arrow (1.057x ➖, 1↑ 6↓)
duckdb / vortex-file-compressed (0.916x ➖, 9↑ 0↓)
duckdb / vortex-compact (1.027x ➖, 0↑ 0↓)
duckdb / parquet (1.001x ➖, 1↑ 0↓)
duckdb / duckdb (1.029x ➖, 0↑ 0↓)
File Size Changes (27 files changed, -0.0% overall, 15↑ 12↓)
Totals:
|
Benchmarks: Clickbench on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.019x ➖, 0↑ 1↓)
datafusion / parquet (1.011x ➖, 0↑ 0↓)
duckdb / vortex-file-compressed (1.019x ➖, 2↑ 3↓)
duckdb / parquet (1.019x ➖, 1↑ 1↓)
duckdb / duckdb (1.012x ➖, 0↑ 2↓)
File Size Changes (107 files changed, +0.0% overall, 56↑ 51↓)
Totals:
|
Benchmarks: TPC-H SF=1 on S3Verdict: No clear signal (environment too noisy confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (1.080x ➖, 0↑ 3↓)
datafusion / vortex-compact (1.148x ➖, 0↑ 3↓)
datafusion / parquet (1.088x ➖, 1↑ 4↓)
duckdb / vortex-file-compressed (1.087x ➖, 0↑ 1↓)
duckdb / vortex-compact (1.105x ➖, 0↑ 0↓)
duckdb / parquet (1.156x ➖, 0↑ 1↓)
|
🚨🚨🚨❌❌❌ SQL BENCHMARK FAILED ❌❌❌🚨🚨🚨Benchmark |
Benchmarks: Appian on NVMEVerdict: No clear signal (low confidence) How to read Verdict and Engines
datafusion / vortex-file-compressed (0.988x ➖, 0↑ 0↓)
datafusion / parquet (0.988x ➖, 0↑ 0↓)
duckdb / vortex-file-compressed (1.001x ➖, 0↑ 0↓)
duckdb / parquet (1.005x ➖, 0↑ 0↓)
duckdb / duckdb (0.999x ➖, 0↑ 0↓)
File Size Changes (4 files changed, -0.0% overall, 2↑ 2↓)
Totals:
|
Benchmarks: CompressionVortex (geomean): 0.998x ➖ How to read Verdict and Engines
unknown / unknown (1.005x ➖, 1↑ 2↓)
|
Rationale for this change
Inspired by @0ax1, trying to see how many dependencies we can remove/simplify