Skip to content

Pass Columnar arguments to scalar functions#6718

Open
gatesn wants to merge 2 commits intongates/execution-argsfrom
ngates/columnar-args
Open

Pass Columnar arguments to scalar functions#6718
gatesn wants to merge 2 commits intongates/execution-argsfrom
ngates/columnar-args

Conversation

@gatesn
Copy link
Contributor

@gatesn gatesn commented Feb 28, 2026

Actually an open question now that I am thinking about aggregates

Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn added changelog/break A breaking API change action/benchmark-sql Trigger SQL benchmarks to run on this PR labels Feb 28, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 28, 2026

Merging this PR will degrade performance by 11.89%

❌ 1 regressed benchmark
✅ 953 untouched benchmarks
⏩ 1466 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation bench_many_nulls[0.9] 482.1 µs 547.2 µs -11.89%

Comparing ngates/columnar-args (db2975f) with ngates/execution-args (759beef)

Open in CodSpeed

Footnotes

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

Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn added the do not merge Pull requests that are not intended to merge label Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action/benchmark-sql Trigger SQL benchmarks to run on this PR changelog/break A breaking API change do not merge Pull requests that are not intended to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant