pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: David Rowley <drowley@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Show Parallel Bitmap Heap Scan worker stats in EXPLAIN ANALYZE
Date: Tue, 09 Jul 2024 00:16:16 +0000
Message-ID: <E1sQyWm-000v23-1S@gemulon.postgresql.org> (raw)

Show Parallel Bitmap Heap Scan worker stats in EXPLAIN ANALYZE

Nodes like Memoize report the cache stats for each parallel worker, so it
makes sense to show the exact and lossy pages in Parallel Bitmap Heap Scan
in a similar way.  Likewise, Sort shows the method and memory used for
each worker.

There was some discussion on whether the leader stats should include the
totals for each parallel worker or not.  I did some analysis on this to
see what other parallel node types do and it seems only Parallel Hash does
anything like this.  All the rest, per what's supported by
ExecParallelRetrieveInstrumentation() are consistent with each other.

Author: David Geier <geidav.pg@gmail.com>
Author: Heikki Linnakangas <hlinnaka@iki.fi>
Author: Donghang Lin <donghanglin@gmail.com>
Author: Alena Rybakina <lena.ribackina@yandex.ru>
Author: David Rowley <dgrowleyml@gmail.com>
Reviewed-by: Dmitry Dolgov <9erthalion6@gmail.com>
Reviewed-by: Michael Christofides <michael@pgmustard.com>
Reviewed-by: Robert Haas <robertmhaas@gmail.com>
Reviewed-by: Dilip Kumar <dilipbalaut@gmail.com>
Reviewed-by: Tomas Vondra <tomas.vondra@enterprisedb.com>
Reviewed-by: Melanie Plageman <melanieplageman@gmail.com>
Reviewed-by: Donghang Lin <donghanglin@gmail.com>
Reviewed-by: Masahiro Ikeda <Masahiro.Ikeda@nttdata.com>
Discussion: https://postgr.es/m/b3d80961-c2e5-38cc-6a32-61886cdf766d%40gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5a1e6df3b84c91957f80b19edb497a5eec83c403

Modified Files
--------------
src/backend/commands/explain.c            |  58 ++++++++++++++---
src/backend/executor/execParallel.c       |   3 +
src/backend/executor/nodeBitmapHeapscan.c | 105 +++++++++++++++++++++++++++---
src/include/executor/nodeBitmapHeapscan.h |   1 +
src/include/nodes/execnodes.h             |  35 ++++++++--
src/tools/pgindent/typedefs.list          |   2 +
6 files changed, 181 insertions(+), 23 deletions(-)



Message-ID: <E1sQyWm-000v23-1S@gemulon.postgresql.org>
Permalink:  ../E1sQyWm-000v23-1S@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1sQyWm-000v23-1S@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: drowley@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Show Parallel Bitmap Heap Scan worker stats in EXPLAIN ANALYZE
  In-Reply-To: <E1sQyWm-000v23-1S@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox