Skip to content

Drop VarBinView's debug validation in new_unchecked - #10584

Merged
robert3005 merged 2 commits into
developfrom
rk/legacy-session-varbinview
Oct 20, 2026
Merged

robert3005 merged 2 commits into
developfrom
rk/legacy-session-varbinview

Conversation

@robert3005

@robert3005 robert3005 commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

VarBinViewData::new_unchecked ran a debug-only validation that executed the validity using legacy_session(). It is deleted. Callers of new_unchecked already promise valid parts, and the checked constructors still validate.

part of #9446

`VarBinViewArray::new_unchecked` validated its parts in debug builds with an
execution context from the hidden global `legacy_session()`. Validate only
when the validity is constant or a canonical host-resident bitmap, so the
debug check never executes anything.

Signed-off-by: Claude <noreply@anthropic.com>

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y1zHpE1K1Y9HvyU4kt24Tq
The debug-only check executed the validity with the global legacy session.
Callers of new_unchecked already promise valid parts, so drop the check
instead of keeping a weaker one.

Signed-off-by: Claude <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y1zHpE1K1Y9HvyU4kt24Tq
@robert3005 robert3005 added the changelog/chore A trivial change label Oct 19, 2026
@codspeed

codspeed Bot commented Oct 19, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 0.93%

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

⚠️ 16 benchmarks spent significant time in system calls

System calls cannot be consistently instrumented, so they are not included in the measure, which understates the real cost. Please switch to the Walltime instrument to accurately measure system calls.

Measurement and system calls

ℹ️ 40 benchmarks have no flame graph

The flamegraph could not be generated because we failed to resolve the symbols in the profiling trace. This happens when we fail to harvest information about language frames.

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

⚡ 3 improved benchmarks
❌ 4 regressed benchmarks
✅ 2401 untouched benchmarks
⏩ 12 skipped benchmarks1

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
❌ ℹ️ Simulation test_scan_filter_to_arrow_array[int_float] 3.5 ms 4.1 ms -13.68%
❌ Simulation isolated_runs[u32, (16, 0)] 78.5 µs 88.4 µs -11.21%
❌ ℹ️ Simulation test_repeated_scan_filter[int_float] 3.4 ms 3.9 ms -11.19%
❌ ℹ️ Simulation test_repeated_scan_filter[int] 2.4 ms 2.7 ms -10.65%
⚡ WallTime bitpack_blocked_compress_avx512 5.6 µs 4.7 µs +19.13%
⚡ WallTime threshold_neon[80] 3.9 µs 3.4 µs +16.6%
⚡ WallTime mul_u16_nonnull_avx2 12.2 µs 11 µs +10.85%

Tip

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


Comparing rk/legacy-session-varbinview (5b51f4c) with develop (31e0724)

Open in CodSpeed

Footnotes

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

@robert3005 robert3005 changed the title Skip executing validity in VarBinView debug validation Drop VarBinView's debug validation in new_unchecked Oct 19, 2026
@robert3005
robert3005 marked this pull request as ready for review October 19, 2026 12:28
@robert3005
robert3005 enabled auto-merge (squash) October 19, 2026 14:57
@robert3005
robert3005 merged commit 7512e99 into develop Oct 20, 2026
129 of 132 checks passed
@robert3005
robert3005 deleted the rk/legacy-session-varbinview branch October 20, 2026 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants