Skip to content

RowFn framework and machinery - #9319

Open
connortsui20 wants to merge 2 commits into
developfrom
ct/row-fn-api
Open

RowFn framework and machinery#9319
connortsui20 wants to merge 2 commits into
developfrom
ct/row-fn-api

Conversation

@connortsui20

@connortsui20 connortsui20 commented Aug 10, 2026

Copy link
Copy Markdown
Member

Tracking Issues: #9129, #9130

Adds the RowFn API, its private batch executor, and focused executor benchmarks.

TODO

@connortsui20 connortsui20 added the changelog/feature A new feature label Aug 10, 2026
Introduce typed row execution with self-contained kernel arguments and explicit execution contracts. Validate decoded lengths once per batch while preserving specialized mixed-constant loops.

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
Cover RowFn executor overhead and strict validity policies with focused microbenchmarks.

Signed-off-by: Connor Tsui <connor.tsui20@gmail.com>
@codspeed-hq

codspeed-hq Bot commented Aug 10, 2026

Copy link
Copy Markdown

Merging this PR will regress 1 benchmark

⚡ 2 improved benchmarks
❌ 1 regressed benchmark
✅ 1941 untouched benchmarks
🆕 16 new benchmarks
⏩ 85 skipped benchmarks1

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation decompress[u64, (1000, 16)] 64 µs 72.1 µs -11.27%
Simulation compact[(4096, 90)] 2 µs 1.8 µs +11.17%
Simulation compact[(16384, 90)] 2.1 µs 1.9 µs +10.35%
🆕 Simulation handrolled_sink_wrapping_add N/A 1.7 ms N/A
🆕 Simulation row_checked_add N/A 2.1 ms N/A
🆕 Simulation row_checked_add_constant N/A 1.6 ms N/A
🆕 Simulation row_checked_add_nullable N/A 2.1 ms N/A
🆕 Simulation row_sink_wrapping_add N/A 2.8 ms N/A
🆕 Simulation row_wrapping_add N/A 1.8 ms N/A
🆕 Simulation row_wrapping_add_constant N/A 1.2 ms N/A
🆕 Simulation row_wrapping_add_nullable N/A 1.8 ms N/A
🆕 Simulation eager_chain[1048576] N/A 25.2 ms N/A
🆕 Simulation eager_chain[65536] N/A 1.9 ms N/A
🆕 Simulation eager[1048576] N/A 9.3 ms N/A
🆕 Simulation eager[65536] N/A 660.8 µs N/A
🆕 Simulation lazy_chain[1048576] N/A 20.5 ms N/A
🆕 Simulation lazy_chain[65536] N/A 1.3 ms N/A
🆕 Simulation lazy[1048576] N/A 9.2 ms N/A
🆕 Simulation lazy[65536] N/A 650.5 µs N/A

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-api (84837ad) with develop (17b190b)

Open in CodSpeed

Footnotes

  1. 85 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.

@connortsui20
connortsui20 marked this pull request as ready for review August 10, 2026 02:58
@connortsui20 connortsui20 changed the title Add the RowFn scalar function framework RowFn API Aug 10, 2026
@connortsui20 connortsui20 changed the title RowFn API RowFn framework and machinery Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant