Repository navigation
Support List/Struct/Map scalars in DuckDB conversion - #10639
Conversation
Merging this PR will degrade performance by 14.19%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Walltime | threshold_neon[8] |
1.3 µs | 2.5 µs | -46.17% |
| ❌ | Walltime | filter_neon[8] |
898 ns | 1,306 ns | -31.24% |
| ❌ | Walltime | threshold_neon[64] |
2.5 µs | 3.4 µs | -25.9% |
| ❌ | Walltime | threshold_neon[80] |
3.4 µs | 4.3 µs | -19.78% |
| ❌ | Walltime | filter_neon[64] |
1.7 µs | 2.1 µs | -17.29% |
| ❌ | Walltime | filter_neon[80] |
2.5 µs | 2.9 µs | -13.6% |
| ❌ ℹ️ | CPU Simulation | test_scan_filter_to_arrow_array[int_float] |
3.5 ms | 4.1 ms | -13.54% |
| ❌ |
CPU Simulation | test_repeated_scan_scalar_at[int_str_float] |
489.1 µs | 552.7 µs | -11.5% |
| ❌ ℹ️ | CPU Simulation | test_repeated_scan_filter[int_float] |
3.4 ms | 3.9 ms | -10.96% |
| ❌ ℹ️ | CPU Simulation | test_repeated_scan_filter[int] |
2.4 ms | 2.7 ms | -10.38% |
| ❌ ℹ️ | CPU Simulation | test_scan_filter_to_arrow_array[int] |
2.5 ms | 2.8 ms | -10.06% |
| ⚡ |
CPU Simulation | density_sweep_dense_runs[0.001] |
61.4 µs | 43.4 µs | +41.63% |
| ⚡ | Walltime | bitpack_blocked_compress_avx2 |
7.6 µs | 6.8 µs | +11.87% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing claude/happy-wozniak-qw3lsw (fe132e5) with develop (d200566)
Footnotes
-
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. ↩
Convert nested Vortex scalars to DuckDB LIST, STRUCT and MAP values via the DuckDB C API value constructors, and map the Vortex Map dtype to a DuckDB MAP logical type. FixedSizeList now shares the List code path. Signed-off-by: Claude <noreply@anthropic.com> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L6XhomwAohFRGEPuLrtUR9
7d5b6ab to
39a7eaf
Compare
Signed-off-by: Claude <noreply@anthropic.com> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L6XhomwAohFRGEPuLrtUR9
Signed-off-by: Claude <noreply@anthropic.com> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L6XhomwAohFRGEPuLrtUR9
Convert nested Vortex scalars to DuckDB LIST, STRUCT and MAP values via the DuckDB C API value constructors, and map the Vortex Map dtype to a DuckDB MAP logical type.