Skip to content

Add List to interleave_kernels benchmark#8980

Merged
Dandandan merged 1 commit intoapache:mainfrom
alamb:alamb/interleave_bench
Dec 11, 2025
Merged

Add List to interleave_kernels benchmark#8980
Dandandan merged 1 commit intoapache:mainfrom
alamb:alamb/interleave_bench

Conversation

@alamb
Copy link
Copy Markdown
Contributor

@alamb alamb commented Dec 10, 2025

Which issue does this PR close?

Rationale for this change

While reviewing #8953 from @asubiotto I noticed there was no benchmark for interleave with ListArray. Let's add some so we can evaluate the performance impact of ttps://github.com//pull/8953 and future changes.

What changes are included in this PR?

Add benchmark for list interleaving

Are these changes tested?

I ran the bechmarks manually

cargo bench --bench interleave_kernels -- list

Are there any user-facing changes?

No

@alamb alamb changed the title Add List benchmark to interleave kernels Add List to interleave_kernels benchmark Dec 10, 2025
@github-actions github-actions bot added the arrow Changes to the arrow crate label Dec 10, 2025
@alamb alamb force-pushed the alamb/interleave_bench branch from aba3ad9 to d61bf3c Compare December 10, 2025 21:02
@alamb alamb marked this pull request as ready for review December 10, 2025 21:24
Copy link
Copy Markdown
Contributor

@asubiotto asubiotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. In the future feel free to push this work onto me, happy to take it off your plate.

@Dandandan Dandandan merged commit 026a260 into apache:main Dec 11, 2025
23 checks passed
@alamb alamb deleted the alamb/interleave_bench branch December 11, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arrow Changes to the arrow crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants