Skip to content

Add Patches lookup benchmarks#7795

Merged
palaska merged 4 commits intodevelopfrom
bp/patches-bench
May 5, 2026
Merged

Add Patches lookup benchmarks#7795
palaska merged 4 commits intodevelopfrom
bp/patches-bench

Conversation

@palaska
Copy link
Copy Markdown
Contributor

@palaska palaska commented May 5, 2026

Isolating benchmarks before this optimization: #7753

palaska added 2 commits May 5, 2026 14:09
Signed-off-by: Baris Palaska <barispalaska@gmail.com>
Signed-off-by: Baris Palaska <barispalaska@gmail.com>
@palaska palaska requested a review from joseph-isaacs May 5, 2026 13:15
@palaska palaska added the changelog/chore A trivial change label May 5, 2026
Comment thread vortex-array/benches/patches_lookup.rs Outdated
Signed-off-by: Baris Palaska <barispalaska@gmail.com>
@palaska palaska enabled auto-merge (squash) May 5, 2026 13:37
@palaska palaska requested a review from joseph-isaacs May 5, 2026 13:37
Comment thread vortex-array/benches/patches_lookup.rs Outdated
Signed-off-by: Baris Palaska <barispalaska@gmail.com>
@palaska palaska added the action/benchmark Trigger full benchmarks to run on this PR label May 5, 2026
@github-actions github-actions Bot removed the action/benchmark Trigger full benchmarks to run on this PR label May 5, 2026
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented May 5, 2026

Merging this PR will improve performance by 11.85%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚡ 1 improved benchmark
✅ 1168 untouched benchmarks
🆕 10 new benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
🆕 Simulation search_index_above_max N/A 266 µs N/A
🆕 Simulation search_index_in_range N/A 264.4 µs N/A
🆕 Simulation search_index_below_min_chunked N/A 1.6 ms N/A
🆕 Simulation search_index_mixed_out_of_range_chunked N/A 1.6 ms N/A
🆕 Simulation search_index_below_min N/A 262 µs N/A
🆕 Simulation search_index_mixed_out_of_range N/A 264.1 µs N/A
🆕 Simulation search_index_in_range_chunked N/A 2.1 ms N/A
🆕 Simulation search_index_full_range_random N/A 264 µs N/A
🆕 Simulation search_index_above_max_chunked N/A 1.9 ms N/A
🆕 Simulation search_index_full_range_random_chunked N/A 1.6 ms N/A
Simulation bitwise_not_vortex_buffer_mut[128] 275.3 ns 246.1 ns +11.85%

Comparing bp/patches-bench (0f8ff55) with develop (a31a0db)

Open in CodSpeed

@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@vortex-data vortex-data deleted a comment from github-actions Bot May 5, 2026
@palaska palaska merged commit 1718eb3 into develop May 5, 2026
83 of 92 checks passed
@palaska palaska deleted the bp/patches-bench branch May 5, 2026 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants