take() on VarBin returns VarBinView by myrrc · Pull Request #9229 · vortex-data/vortex

2 benchmarks regressed

⚡ 4 improved benchmarks
❌ 2 regressed benchmarks
✅ 1923 untouched benchmarks
🆕 1 new benchmark
⏩ 51 skipped benchmarks¹

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

	Mode	Benchmark	`BASE`	`HEAD`	Efficiency
❌	Simulation	`fsl_large`	200.6 µs	254.3 µs	-21.14%
❌	Simulation	`compress_fsst[(10000, 64, 8)]`	9.6 ms	10.8 ms	-11.3%
⚡	Simulation	`compact_sliced[(4096, 90)]`	1.8 µs	1.6 µs	+12.16%
⚡	Simulation	`decompress[u64, (1000, 16)]`	73.1 µs	65.1 µs	+12.16%
⚡	Simulation	`chunked_varbin_opt_to_varbinview_builder[(10, 100)]`	870.4 µs	779.8 µs	+11.62%
⚡	Simulation	`compact_sliced[(16384, 90)]`	2 µs	1.8 µs	+11.2%
🆕	Simulation	`take_varbin`	N/A	1.1 ms	N/A

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

_{Comparing myrrc/varbin-take-zero-copy (d8d798a) with develop (c1ae775)}

51 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. ↩