Skip to content

Bigger split sizes for aggregate scans - #9055

Open
myrrc wants to merge 3 commits into
zonemap-stats-layoutfrom
myrrc/zonemap-stats-aggregate-scan-splits
Open

Bigger split sizes for aggregate scans#9055
myrrc wants to merge 3 commits into
zonemap-stats-layoutfrom
myrrc/zonemap-stats-aggregate-scan-splits

Conversation

@myrrc

@myrrc myrrc commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Aggregate splits reading from zone maps don't decode columns so current split
size creates overhead for them (clickbench q6, vortex-bench q3). Set 4 big
splits per code to mitigate this overhead

@myrrc myrrc added the changelog/performance A performance improvement label Jul 29, 2026
@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from d5c10cd to b414ebe Compare July 29, 2026 11:38
@codspeed-hq

codspeed-hq Bot commented Jul 29, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 1841 untouched benchmarks
⏩ 55 skipped benchmarks1


Comparing myrrc/zonemap-stats-aggregate-scan-splits (62d1e8a) with zonemap-stats-layout (615d8ce)

Open in CodSpeed

Footnotes

  1. 55 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.

@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from b414ebe to d5c10cd Compare July 29, 2026 14:47
@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from d5c10cd to 113e69b Compare July 29, 2026 15:01
@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from 113e69b to cb9a26f Compare July 29, 2026 16:49
@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from cb9a26f to b6c85f0 Compare July 29, 2026 17:00
@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from b6c85f0 to 337bfe9 Compare July 29, 2026 17:19
@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from 337bfe9 to acc8d26 Compare July 29, 2026 17:28
@myrrc
myrrc changed the base branch from zonemap-stats-duckdb to zonemap-stats-layout July 29, 2026 17:30
@myrrc
myrrc force-pushed the myrrc/zonemap-stats-aggregate-scan-splits branch from acc8d26 to 62d1e8a Compare July 29, 2026 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/performance A performance improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant