Skip to content

bench(parquet): add Sbbf check/insert benchmarks#10041

Merged
etseidl merged 1 commit into
apache:mainfrom
dmatth1:bloom-filter-benches
May 31, 2026
Merged

bench(parquet): add Sbbf check/insert benchmarks#10041
etseidl merged 1 commit into
apache:mainfrom
dmatth1:bloom-filter-benches

Conversation

@dmatth1
Copy link
Copy Markdown
Contributor

@dmatth1 dmatth1 commented May 30, 2026

Adds bench_check and bench_insert benchmarks forSbbf::{check,insert}. Originally benchmarks were part of #10011 but were split out to follow Contributing guidelines

Are these changes tested?

Benchmarks compiled and ran using cargo bench -p parquet --bench bloom_filter.

Are there any user-facing changes?

No.

Adds `bench_check` (miss/hit x 128KiB/2MiB/32MiB cache regimes) and
`bench_insert`, both exercising the public `Sbbf::{check,insert}` API.
The miss set is constructed disjoint from the inserted set (high bit of
byte 0 flipped) so the miss path is genuinely a miss, not a
birthday-paradox near-miss.

Precursor to the Sbbf autovectorization change (apache#10011) so before/after
numbers can be reproduced against benchmarks already on main.
@github-actions github-actions Bot added the parquet Changes to the parquet crate label May 30, 2026
@dmatth1 dmatth1 changed the title bench(parquet): add Sbbf check/insert benchmarks across cache regimes bench(parquet): add Sbbf check/insert benchmarks May 30, 2026
Copy link
Copy Markdown
Contributor

@etseidl etseidl left a comment

Choose a reason for hiding this comment

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

Thanks @dmatth1!

@etseidl etseidl merged commit 511ad06 into apache:main May 31, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parquet Changes to the parquet crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants