Implement the RowFn execution backend - #9353
Conversation
Merging this PR will degrade performance by 26.3%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | decode_varbin[(1000, 4)] |
60.3 µs | 97.4 µs | -38.06% |
| ❌ | WallTime | words_gather_scalar[65536] |
8.2 µs | 9.4 µs | -12.31% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing ct/row-fn-framework (6ddd036) with ct/row-fn-types (707d106)2
Footnotes
-
89 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. ↩
-
No successful run was found on
ct/row-fn-types(c02fe9d) during the generation of this report, so 0f50f99 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
92e8d51 to
2ceb889
Compare
2ceb889 to
899b19a
Compare
899b19a to
6fdcd8e
Compare
583e721 to
71ad756
Compare
f677e59 to
91f1511
Compare
91f1511 to
4a1d66c
Compare
4a1d66c to
944b540
Compare
944b540 to
8f318a3
Compare
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
8f318a3 to
6ddd036
Compare
Rationale for this change
RowFnover Vortex arrays #9130RowFnAPI #9129Implements execution for the contracts in #9386.
What changes are included in this PR?
Adds batch validity and constant handling, dense and valid-only strategies, prepared row loops, sink execution, reducer handling, and output validation. It completes the
execute_rowspath in the blanket adapter introduced by #9386.What APIs are changed? Are there any user-facing changes?
Adds execution internals within the existing
unstable_row_fnsboundary. There are no stable API changes.