perf(array): hoist per-row work out of append_varbinview by robert3005 · Pull Request #8994 · vortex-data/vortex

Performance Regression: -20.83%

❌ 1 regressed benchmark
✅ 1848 untouched benchmarks
🆕 8 new benchmarks
⏩ 46 skipped benchmarks¹

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

	Mode	Benchmark	`BASE`	`HEAD`	Efficiency
❌	Simulation	`decode_varbin[(1000, 8)]`	27 µs	34.1 µs	-20.83%
🆕	Simulation	`heap_all_valid[4096]`	N/A	395.5 µs	N/A
🆕	Simulation	`heap_with_nulls[4096]`	N/A	416.3 µs	N/A
🆕	Simulation	`inlined_all_valid[4096]`	N/A	236.5 µs	N/A
🆕	Simulation	`inlined_with_nulls[4096]`	N/A	304.9 µs	N/A
🆕	Simulation	`heap_all_valid[16384]`	N/A	1.5 ms	N/A
🆕	Simulation	`heap_with_nulls[16384]`	N/A	1.6 ms	N/A
🆕	Simulation	`inlined_all_valid[16384]`	N/A	905 µs	N/A
🆕	Simulation	`inlined_with_nulls[16384]`	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 claude/varbin-builder-bulk-append (8110b99) with develop (b4f8d34)}

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