NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2bb1f14

committed

Make bitmap heap scans show exact/lossy block info in EXPLAIN ANALYZE.

Etsuro Fujita

1 parentc3ccc9e commit2bb1f14Copy full SHA for 2bb1f14

File tree

3 files changed

+43

-1

lines changed

src
- backend
  - commands
    - explain.c
  - executor
    - nodeBitmapHeapscan.c
- include/nodes
  - execnodes.h

3 files changed

+43

-1

lines changed

`‎src/backend/commands/explain.c`

Lines changed: 32 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,8 @@ static void show_sort_group_keys(PlanState planstate, const char qlabel,`
`85`	`85`	`Listancestors,ExplainStatees);`
`86`	`86`	`staticvoidshow_sort_info(SortStatesortstate,ExplainStatees);`
`87`	`87`	`staticvoidshow_hash_info(HashStatehashstate,ExplainStatees);`
	`88`	`+staticvoidshow_tidbitmap_info(BitmapHeapScanState*planstate,`
	`89`	`+ExplainState*es);`
`88`	`90`	`staticvoidshow_instrumentation_count(constchar*qlabel,intwhich,`
`89`	`91`	`PlanStateplanstate,ExplainStatees);`
`90`	`92`	`staticvoidshow_foreignscan_info(ForeignScanStatefsstate,ExplainStatees);`
`@@ -1250,7 +1252,13 @@ ExplainNode(PlanState planstate, List ancestors,`
`1250`	`1252`	`if (((BitmapHeapScan*)plan)->bitmapqualorig)`
`1251`	`1253`	`show_instrumentation_count("Rows Removed by Index Recheck",2,`
`1252`	`1254`	`planstate,es);`
`1253`		`-/* FALL THRU */`
	`1255`	`+show_scan_qual(plan->qual,"Filter",planstate,ancestors,es);`
	`1256`	`+if (plan->qual)`
	`1257`	`+show_instrumentation_count("Rows Removed by Filter",1,`
	`1258`	`+planstate,es);`
	`1259`	`+if (es->analyze)`
	`1260`	`+show_tidbitmap_info((BitmapHeapScanState*)planstate,es);`
	`1261`	`+break;`
`1254`	`1262`	`caseT_SeqScan:`
`1255`	`1263`	`caseT_ValuesScan:`
`1256`	`1264`	`caseT_CteScan:`
`@@ -1878,6 +1886,29 @@ show_hash_info(HashState hashstate, ExplainState es)`
`1878`	`1886`	`}`
`1879`	`1887`	`}`
`1880`	`1888`
	`1889`	`+/*`
	`1890`	`+ * If it's EXPLAIN ANALYZE, show exact/lossy pages for a BitmapHeapScan node`
	`1891`	`+ */`
	`1892`	`+staticvoid`
	`1893`	`+show_tidbitmap_info(BitmapHeapScanStateplanstate,ExplainStatees)`
	`1894`	`+{`
	`1895`	`+if (es->format!=EXPLAIN_FORMAT_TEXT)`
	`1896`	`+{`
	`1897`	`+ExplainPropertyLong("Exact Heap Blocks",planstate->exact_pages,es);`
	`1898`	`+ExplainPropertyLong("Lossy Heap Blocks",planstate->lossy_pages,es);`
	`1899`	`+}`
	`1900`	`+else`
	`1901`	`+{`
	`1902`	`+appendStringInfoSpaces(es->str,es->indent*2);`
	`1903`	`+appendStringInfoString(es->str,"Heap Blocks:");`
	`1904`	`+if (planstate->exact_pages>0)`
	`1905`	`+appendStringInfo(es->str," exact=%ld",planstate->exact_pages);`
	`1906`	`+if (planstate->lossy_pages>0)`
	`1907`	`+appendStringInfo(es->str," lossy=%ld",planstate->lossy_pages);`
	`1908`	`+appendStringInfoChar(es->str,'\n');`
	`1909`	`+}`
	`1910`	`+}`
	`1911`	`+`
`1881`	`1912`	`/*`
`1882`	`1913`	`* If it's EXPLAIN ANALYZE, show instrumentation information for a plan node`
`1883`	`1914`	`*`

`‎src/backend/executor/nodeBitmapHeapscan.c`

Lines changed: 7 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -170,6 +170,11 @@ BitmapHeapNext(BitmapHeapScanState *node)`
`170`	`170`	`*/`
`171`	`171`	`bitgetpage(scan,tbmres);`
`172`	`172`
	`173`	`+if (tbmres->ntuples >=0)`
	`174`	`+node->exact_pages++;`
	`175`	`+else`
	`176`	`+node->lossy_pages++;`
	`177`	`+`
`173`	`178`	`/*`
`174`	`179`	`* Set rs_cindex to first slot to examine`
`175`	`180`	`*/`
`@@ -553,6 +558,8 @@ ExecInitBitmapHeapScan(BitmapHeapScan node, EState estate, int eflags)`
`553`	`558`	`scanstate->tbm=NULL;`
`554`	`559`	`scanstate->tbmiterator=NULL;`
`555`	`560`	`scanstate->tbmres=NULL;`
	`561`	`+scanstate->exact_pages=0;`
	`562`	`+scanstate->lossy_pages=0;`
`556`	`563`	`scanstate->prefetch_iterator=NULL;`
`557`	`564`	`scanstate->prefetch_pages=0;`
`558`	`565`	`scanstate->prefetch_target=0;`

`‎src/include/nodes/execnodes.h`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1338,6 +1338,8 @@ typedef struct BitmapIndexScanState`
`1338`	`1338`	`*tbm bitmap obtained from child index scan(s)`
`1339`	`1339`	`*tbmiterator iterator for scanning current pages`
`1340`	`1340`	`*tbmres current-page data`
	`1341`	`+ *exact_pages total number of exact pages retrieved`
	`1342`	`+ *lossy_pages total number of lossy pages retrieved`
`1341`	`1343`	`*prefetch_iterator iterator for prefetching ahead of current page`
`1342`	`1344`	`*prefetch_pages # pages prefetch iterator is ahead of current`
`1343`	`1345`	`*prefetch_target target prefetch distance`
`@@ -1350,6 +1352,8 @@ typedef struct BitmapHeapScanState`
`1350`	`1352`	`TIDBitmap*tbm;`
`1351`	`1353`	`TBMIterator*tbmiterator;`
`1352`	`1354`	`TBMIterateResult*tbmres;`
	`1355`	`+longexact_pages;`
	`1356`	`+longlossy_pages;`
`1353`	`1357`	`TBMIterator*prefetch_iterator;`
`1354`	`1358`	`intprefetch_pages;`
`1355`	`1359`	`intprefetch_target;`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit2bb1f14

File tree

3 files changed

3 files changed

`‎src/backend/commands/explain.c`

`‎src/backend/executor/nodeBitmapHeapscan.c`

`‎src/include/nodes/execnodes.h`

0 commit comments