Commit9f9135d

committed

Instrument index-only scans to count heap fetches performed.

Patch by me; review by Tom Lane, Jeff Davis, and Peter Geoghegan.

1 parent0814677 commit9f9135dCopy full SHA for 9f9135d

File tree

-0

lines changed

-0

lines changed

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1012,6 +1012,9 @@ ExplainNode(PlanState planstate, List ancestors,`
`1012`	`1012`	`if (plan->qual)`
`1013`	`1013`	`show_instrumentation_count("Rows Removed by Filter",1,`
`1014`	`1014`	`planstate,es);`
	`1015`	`+if (es->analyze)`
	`1016`	`+ExplainPropertyLong("Heap Fetches",`
	`1017`	`+((IndexOnlyScanState*)planstate)->ioss_HeapFetches,es);`
`1015`	`1018`	`break;`
`1016`	`1019`	`caseT_BitmapIndexScan:`
`1017`	`1020`	`show_scan_qual(((BitmapIndexScan*)plan)->indexqualorig,`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -90,6 +90,7 @@ IndexOnlyNext(IndexOnlyScanState *node)`
`90`	`90`	`/*`
`91`	`91`	`* Rats, we have to visit the heap to check visibility.`
`92`	`92`	`*/`
	`93`	`+node->ioss_HeapFetches++;`
`93`	`94`	`tuple=index_fetch_heap(scandesc);`
`94`	`95`	`if (tuple==NULL)`
`95`	`96`	`continue;/* no visible tuple, try next index entry */`
`@@ -346,6 +347,7 @@ ExecInitIndexOnlyScan(IndexOnlyScan node, EState estate, int eflags)`
`346`	`347`	`indexstate=makeNode(IndexOnlyScanState);`
`347`	`348`	`indexstate->ss.ps.plan= (Plan*)node;`
`348`	`349`	`indexstate->ss.ps.state=estate;`
	`350`	`+indexstate->ioss_HeapFetches=0;`
`349`	`351`
`350`	`352`	`/*`
`351`	`353`	`* Miscellaneous initialization`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1260,6 +1260,7 @@ typedef struct IndexScanState`
`1260`	`1260`	`*RelationDesc index relation descriptor`
`1261`	`1261`	`*ScanDesc index scan descriptor`
`1262`	`1262`	`*VMBuffer buffer in use for visibility map testing, if any`
	`1263`	`+ *HeapFetches number of tuples we were forced to fetch from heap`
`1263`	`1264`	`* ----------------`
`1264`	`1265`	`*/`
`1265`	`1266`	`typedefstructIndexOnlyScanState`
`@@ -1277,6 +1278,7 @@ typedef struct IndexOnlyScanState`
`1277`	`1278`	`Relationioss_RelationDesc;`
`1278`	`1279`	`IndexScanDescioss_ScanDesc;`
`1279`	`1280`	`Bufferioss_VMBuffer;`
	`1281`	`+longioss_HeapFetches;`
`1280`	`1282`	`}IndexOnlyScanState;`
`1281`	`1283`
`1282`	`1284`	`/* ----------------`

Comments

(0)