Skip to content

Fix heap blocks bug add test coverage#59

Open
lfittl wants to merge 2 commits intomasterfrom
fix-heap-blocks-bug-add-test-coverage
Open

Fix heap blocks bug add test coverage#59
lfittl wants to merge 2 commits intomasterfrom
fix-heap-blocks-bug-add-test-coverage

Conversation

@lfittl
Copy link
Copy Markdown
Owner

@lfittl lfittl commented Apr 5, 2026

No description provided.

lfittl added 2 commits April 5, 2026 11:20
Fix the missing accumulation of "Heap Blocks" from parallel query workers
to the leader, causing EXPLAIN (ANALYZE) to only show the leader statistics,
significantly undercounting the true value.

Additionally, add a regression test covering EXPLAIN (ANALYZE) of a
Parallel Bitmap Heap Scan, which previously was not tested at all.

Author: Lukas Fittl <lukas@fittl.com>
Reviewed-by:
Discussion
The functions dealing with copying back parallel worker instrumentation
such as ExecIndexOnlyScanRetrieveInstrumentation were not exercised
at all in the regression tests, leading to a gap in coverage. Add a
query that verifies we correctly copy back "Index Searches" for
EXPLAIN ANALYZE of a Parallel Index Only Scan.

Reported-by: Andres Freund <andres@anarazel.de>
Author: Lukas Fittl <lukas@fittl.com>
Discussion:
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