Skip to content

use full column set on empty post-filter projection list in duckdb #6831

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
myrrc wants to merge 1 commit into
base: develop
Choose a base branch
from
+294 −95

use full column set on empty post-filter projection list in duckdb

d764aed
Select commit
Loading
Failed to load commit list.
Open

use full column set on empty post-filter projection list in duckdb #6831

use full column set on empty post-filter projection list in duckdb
d764aed
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis succeeded Mar 6, 2026

Performance Gate Passed

⚡ 3 improved benchmarks
✅ 977 untouched benchmarks
🆕 2 new benchmarks
⏩ 1466 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
⚡ Simulation take_map[(0.1, 0.5)] 2.6 ms 2.1 ms +23.63%
⚡ Simulation take_map[(0.1, 0.1)] 1,007.5 µs 908.9 µs +10.84%
⚡ Simulation take_map[(0.1, 1.0)] 4.2 ms 3.5 ms +20.81%
🆕 Simulation sequence_compress_u32 N/A 5.2 ms N/A
🆕 Simulation sequence_decompress_u32 N/A 4.1 ms N/A

Comparing myrrc/fix-empty-projection-list-duckdb (d764aed) with develop (5d6a3c8)2

Open in CodSpeed

Footnotes

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

  2. No successful run was found on develop (a8b340a) during the generation of this report, so 5d6a3c8 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

View more details on CodSpeed HQ