Skip to content

Pre-allocate data buffer in VarBin::take #4749

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
AdamGS merged 2 commits into from
Sep 25, 2025

CR comments and some other changes

2e90821
Select commit
Loading
Failed to load commit list.
Merged

Pre-allocate data buffer in VarBin::take #4749

CR comments and some other changes
2e90821
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Sep 25, 2025 in 0s

Performance Gate Passed

Summary

⚡ 1 improvement
✅ 1155 untouched
⏩ 292 skipped1

Benchmarks breakdown

Benchmark BASE HEAD Change
⚡ varbin_non_null 156 µs 123.1 µs +26.69%

Footnotes

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