Add variable-arity sequence tables - #974
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## codex/vectorized-free-join-on-962 #974 +/- ##
=====================================================================
+ Coverage 87.27% 87.56% +0.28%
=====================================================================
Files 96 97 +1
Lines 34407 36183 +1776
=====================================================================
+ Hits 30030 31683 +1653
- Misses 4377 4500 +123 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will improve performance by 5.12%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ⚡ | Simulation | tests[eggcc-2mm] |
8.3 s | 7.9 s | +5.12% |
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 codex/inline-containers-01-sequence-table (1b22eca) with codex/vectorized-free-join-on-962 (f93fb49)
Footnotes
-
227 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
SequenceTable, aSortedWritesTableanalogue with variable-length keys and a fixed-width non-key value tail.Subset, version, merge, and generation semantics using packed values plus cumulative row offsets.Stack
Draft, stacked on #966. This is PR 1 of 12 in the sequence-backed container stack.
Validation
Validated on the complete 12-commit stack:
cargo nextest run --workspace: 1,362 tests passedcargo test --workspace --doccargo clippy --workspace --all-targets -- -D warningscargo fmt --all -- --check