Skip to content

performance[vortex-array]: don't call is_valid to count bytes in varbinview #5814

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
asubiotto merged 1 commit into from
Dec 23, 2025

performance[vortex-array]: don't call is_valid to count bytes in varb…

7fda16d
Select commit
Loading
Failed to load commit list.
Merged

performance[vortex-array]: don't call is_valid to count bytes in varbinview #5814

performance[vortex-array]: don't call is_valid to count bytes in varb…
7fda16d
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Dec 23, 2025 in 0s

Performance Gate Passed

Summary

⚡ 4 improvements
✅ 1250 untouched
⏩ 623 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Efficiency
⚡ compact_sliced[(4096, 90)] 547.8 µs 466 µs +17.56%
⚡ compact_sliced[(16384, 90)] 2.1 ms 1.8 ms +17.98%
⚡ compact_sliced[(4096, 10)] 641.4 µs 559.6 µs +14.61%
⚡ compact_sliced[(16384, 10)] 2.5 ms 2.2 ms +14.98%

Footnotes

  1. 623 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. ↩

View more details on CodSpeed HQ