From ad9254aee09917f31f8b32af74e06ebef5032aae Mon Sep 17 00:00:00 2001 From: Peter Geoghegan Date: Sun, 22 Mar 2026 02:36:57 -0400 Subject: [PATCH v17 01/18] Track index-only scan heap fetches using IndexScanInstrumentation. An upcoming commit that adds interfaces that enable index prefetching needs to perform all heap fetches at the table AM layer. The core executor will be left without any way to count the heap fetches itself, so push it down into IndexScanInstrumentation.nheapfetches. Also rename show_indexsearches_info to show_indexscan_info, since the function now reports on more than just index searches. Follow-up to commit f026fbf0, which made IndexScanInstrumentation a pointer in executor scan nodes. Author: Peter Geoghegan Reviewed-By: Andres Freund Discussion: https://postgr.es/m/CAH2-Wz=g=JTSyDB4UtB5su2ZcvsS7VbP+ZMvvaG6ABoCb+s8Lw@mail.gmail.com --- src/include/executor/instrument_node.h | 5 +++++ src/backend/commands/explain.c | 23 +++++++++++++---------- src/backend/executor/nodeIndexonlyscan.c | 3 ++- 3 files changed, 20 insertions(+), 11 deletions(-) diff --git a/src/include/executor/instrument_node.h b/src/include/executor/instrument_node.h index 8847d7f94..1bddd0377 100644 --- a/src/include/executor/instrument_node.h +++ b/src/include/executor/instrument_node.h @@ -48,6 +48,11 @@ typedef struct IndexScanInstrumentation { /* Index search count (incremented with pgstat_count_index_scan call) */ uint64 nsearches; + + /* + * heap blocks fetched counts (incremented during index-only scans) + */ + uint64 nheapfetches; } IndexScanInstrumentation; /* diff --git a/src/backend/commands/explain.c b/src/backend/commands/explain.c index e4b70166b..08b9defd9 100644 --- a/src/backend/commands/explain.c +++ b/src/backend/commands/explain.c @@ -136,7 +136,7 @@ static void show_recursive_union_info(RecursiveUnionState *rstate, static void show_memoize_info(MemoizeState *mstate, List *ancestors, ExplainState *es); static void show_hashagg_info(AggState *aggstate, ExplainState *es); -static void show_indexsearches_info(PlanState *planstate, ExplainState *es); +static void show_indexscan_info(PlanState *planstate, ExplainState *es); static void show_tidbitmap_info(BitmapHeapScanState *planstate, ExplainState *es); static void show_instrumentation_count(const char *qlabel, int which, @@ -1974,7 +1974,7 @@ ExplainNode(PlanState *planstate, List *ancestors, if (plan->qual) show_instrumentation_count("Rows Removed by Filter", 1, planstate, es); - show_indexsearches_info(planstate, es); + show_indexscan_info(planstate, es); break; case T_IndexOnlyScan: show_scan_qual(((IndexOnlyScan *) plan)->indexqual, @@ -1988,15 +1988,12 @@ ExplainNode(PlanState *planstate, List *ancestors, if (plan->qual) show_instrumentation_count("Rows Removed by Filter", 1, planstate, es); - if (es->analyze) - ExplainPropertyFloat("Heap Fetches", NULL, - planstate->instrument->ntuples2, 0, es); - show_indexsearches_info(planstate, es); + show_indexscan_info(planstate, es); break; case T_BitmapIndexScan: show_scan_qual(((BitmapIndexScan *) plan)->indexqualorig, "Index Cond", planstate, ancestors, es); - show_indexsearches_info(planstate, es); + show_indexscan_info(planstate, es); break; case T_BitmapHeapScan: show_scan_qual(((BitmapHeapScan *) plan)->bitmapqualorig, @@ -3860,15 +3857,16 @@ show_hashagg_info(AggState *aggstate, ExplainState *es) } /* - * Show the total number of index searches for a + * Show index scan related executor instrumentation for a * IndexScan/IndexOnlyScan/BitmapIndexScan node */ static void -show_indexsearches_info(PlanState *planstate, ExplainState *es) +show_indexscan_info(PlanState *planstate, ExplainState *es) { Plan *plan = planstate->plan; SharedIndexScanInstrumentation *SharedInfo = NULL; - uint64 nsearches = 0; + uint64 nsearches = 0, + nheapfetches = 0; if (!es->analyze) return; @@ -3889,6 +3887,7 @@ show_indexsearches_info(PlanState *planstate, ExplainState *es) IndexOnlyScanState *indexstate = ((IndexOnlyScanState *) planstate); nsearches = indexstate->ioss_Instrument->nsearches; + nheapfetches = indexstate->ioss_Instrument->nheapfetches; SharedInfo = indexstate->ioss_SharedInfo; break; } @@ -3912,9 +3911,13 @@ show_indexsearches_info(PlanState *planstate, ExplainState *es) IndexScanInstrumentation *winstrument = &SharedInfo->winstrument[i]; nsearches += winstrument->nsearches; + nheapfetches += winstrument->nheapfetches; } } + if (nodeTag(plan) == T_IndexOnlyScan) + ExplainPropertyUInteger("Heap Fetches", NULL, nheapfetches, es); + ExplainPropertyUInteger("Index Searches", NULL, nsearches, es); } diff --git a/src/backend/executor/nodeIndexonlyscan.c b/src/backend/executor/nodeIndexonlyscan.c index 9eab81fd1..1b89570a2 100644 --- a/src/backend/executor/nodeIndexonlyscan.c +++ b/src/backend/executor/nodeIndexonlyscan.c @@ -166,7 +166,8 @@ IndexOnlyNext(IndexOnlyScanState *node) /* * Rats, we have to visit the heap to check visibility. */ - InstrCountTuples2(node, 1); + if (node->ioss_Instrument) + node->ioss_Instrument->nheapfetches++; if (!index_fetch_heap(scandesc, node->ioss_TableSlot)) continue; /* no visible tuple, try next index entry */ -- 2.53.0