Skip to content

Optimize validity checks by resolving to masks upfront#8306

Closed
joseph-isaacs wants to merge 4 commits into
developfrom
claude/audit-per-row-accessors-ln5lho
Closed

Optimize validity checks by resolving to masks upfront#8306
joseph-isaacs wants to merge 4 commits into
developfrom
claude/audit-per-row-accessors-ln5lho

perf: hoist validity/ctx out of display and TUI per-row loops

2c2f634
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis failed Jun 9, 2026 in 0s

3 benchmarks regressed

⚠️ 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.

⚡ 6 improved benchmarks
❌ 3 regressed benchmarks
✅ 1504 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation varbinview_zip_block_mask 2.9 ms 3.7 ms -21.57%
Simulation encode_varbin[(1000, 2)] 143.1 µs 164.1 µs -12.77%
Simulation varbinview_zip_fragmented_mask 6.1 ms 6.9 ms -11.3%
Simulation extend_from_array_non_zctl_overlapping[(10000, 8)] 4.7 ms 2.3 ms ×2
Simulation extend_from_array_non_zctl_overlapping[(1000, 8)] 527.9 µs 287.6 µs +83.56%
Simulation rebuild_naive 142.1 µs 104.9 µs +35.46%
Simulation extend_from_array_non_zctl_overlapping[(1000, 32)] 1,023.8 µs 782.9 µs +30.76%
Simulation extend_from_array_zctl[(10000, 8)] 2.5 ms 2.1 ms +17.04%
Simulation extend_from_array_zctl[(1000, 8)] 291 µs 254.7 µs +14.25%

Tip

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


Comparing claude/audit-per-row-accessors-ln5lho (2c2f634) with develop (1b19ac9)

Open in CodSpeed