Use packed arena tries for generic joins - #959
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## codex/packed-join-abstractions #959 +/- ##
=================================================================
Coverage 87.13% 87.13%
=================================================================
Files 96 96
Lines 31667 33023 +1356
=================================================================
+ Hits 27592 28775 +1183
- Misses 4075 4248 +173 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Merging this PR will degrade performance by 7.82%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | tests[python_array_optimize] |
1.2 s | 1.3 s | -9.53% |
| ❌ | Simulation | tests[eggcc-2mm] |
7.7 s | 8.4 s | -7.65% |
| ❌ | Simulation | tests[typeinfer] |
117.5 ms | 125.3 ms | -6.24% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing ezr-better-parallel-packed-trie (3cfa98a) with codex/packed-join-abstractions (80431f5)
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. ↩
c7fc1d7 to
9eab976
Compare
9eab976 to
3cfa98a
Compare
Stacked on the packed-node and positioned-index abstractions PR.
This is the focused packed-executor integration:
The DVO policy is isolated below the abstraction layer, while demand-driven scheduling is isolated in the PR above this one.
Validation: