Skip to content

Avoid reading fields for empty projections - #9365

Merged
joseph-isaacs merged 0 commit into
ji/vx-plan-cifrom
ji/plan-count-star
Aug 12, 2026
Merged

Avoid reading fields for empty projections#9365
joseph-isaacs merged 0 commit into
ji/vx-plan-cifrom
ji/plan-count-star

Conversation

@joseph-isaacs

@joseph-isaacs joseph-isaacs commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Superseded

This PR is intentionally empty and closed. The COUNT(*) fix was folded into the PRs that own each part of the behavior:

The all-in-one benchmark PR #9342 contains the same distributed history.

@codspeed-hq

codspeed-hq Bot commented Aug 11, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 19.44%

⚡ 1 improved benchmark
✅ 1961 untouched benchmarks
⏩ 89 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation cold_misaligned[(64, 256)] 5.2 ms 4.4 ms +19.44%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing ji/plan-count-star (a836fa0) with ji/vx-plan-ci (cecbc8f)

Open in CodSpeed

Footnotes

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

@joseph-isaacs
joseph-isaacs merged commit ec64cba into ji/vx-plan-ci Aug 12, 2026
1 check passed
@joseph-isaacs
joseph-isaacs deleted the ji/plan-count-star branch August 12, 2026 09:55

Copy link
Copy Markdown
Contributor Author

Folded this fix into the stack at the owning boundaries: #9166 handles the zero-partition expression pushdown, #9240 handles the childless Pack split endpoint, and #9244 contains the end-to-end zero-read regression. This branch now matches #9341 exactly, so this PR has no remaining diff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant