chore: remove cross-implementation benchmarks and rewrite README - #25
Conversation
Signed-off-by: Francesco Gargiulo <gargiulo.fr@gmail.com>
Drop the Rust+C++ (submodule) + Python uv-workspace harness under benchmarks/onpair-bench, keeping only the codspeed benches under benches/. Removes the now-dead root pyproject.toml/uv.lock (their only workspace member was onpair-bench), the sole .gitmodules entry, and the pointless Python/uv .gitignore section. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Rewrite the project README as the front door for engine integrators: a concise overview (independent per-row access, fast index-lookup decode), a Why OnPair summary, a benchmarks placeholder, a verified quick-start example, and high- vs low-cardinality usage guidance. Add crates.io, docs.rs, CI, license, and MSRV badges. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Merging this PR will improve performance by 38.92%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | WallTime | decompress_all[12] |
749 µs | 465.4 µs | +60.93% |
| ⚡ | WallTime | decompress_all[16] |
764 µs | 501.6 µs | +52.31% |
| ⚡ | WallTime | decompress_all[("p_name", 12)] |
1,137.5 µs | 769.9 µs | +47.75% |
| ⚡ | WallTime | decompress_all[("l_comment", 12)] |
16.9 ms | 11.7 ms | +44.29% |
| ⚡ | WallTime | decompress_all[("o_comment", 12)] |
15.1 ms | 10.7 ms | +41.71% |
| ⚡ | WallTime | decompress_all[("p_name", 16)] |
936.5 µs | 697.3 µs | +34.3% |
| ⚡ | WallTime | decompress_all[("o_comment", 16)] |
14.9 ms | 12.4 ms | +20.04% |
| ⚡ | WallTime | decompress_all[("l_comment", 16)] |
15.8 ms | 13.6 ms | +16.24% |
Tip
Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.
Comparing chore/repo-maintenance (e4a24c4) with develop (5534b59)
Footnotes
-
2 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. ↩
|
|
Summary
Repo maintenance ahead of the next release: removes the standalone
cross-implementation benchmarking setup and its tooling, rewrites the README,
and fixes CI. No public API or runtime behavior changes.
Changes
Removed
benchmarks/onpair-bench/(Python corpus/runner, C++ bench, standalone Rust bench) and its git submodule.
examples/bench_tpch.rs).pyproject.toml,uv.lock).benches/(tpch,clickbench) are retained.Docs
README.md.CI
nonexistent cache step).
Tests
Notes
onpair0.1.1 crate.