Re: EXPLAIN: showing ReadStream / prefetch stats

From: Tomas Vondra <tomas(at)vondra(dot)me>
To: Lukas Fittl <lukas(at)fittl(dot)com>
Cc: Melanie Plageman <melanieplageman(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: EXPLAIN: showing ReadStream / prefetch stats
Date: 2026-04-07 12:36:01
Message-ID: 72391376-967e-41b8-806b-41f9efea9a97@vondra.me
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

I've pushed the first two parts earlier today. Here's a v13 with the
remaining pieces rebased on top of that, with updated commit messages
and some additional fixes.

1) There was a bogus else-if in explain.c that would make it impossible
to enable "wal" and "io" at the same time.

2) Replaced one of the "typedef IOStats" with just forward declaration,
to prevent possible conflicting typedefs in the future (even if that
wasn't an issue now).

3) Used (es_instrument & INSTRUMENT_IO) more consistently. A couple
places in the executor still checked just es_instrument, and so would
collect stats even if not needed. Be consistent.

4) Various comment additions / improvements.

I intend to push this later today, after giving it one more round of
review etc.

regards

--
Tomas Vondra

Attachment Content-Type Size
v13-0001-Switch-EXPLAIN-to-unaligned-output-for-json-xml-.patch text/x-patch 13.4 KB
v13-0002-Add-EXPLAIN-IO-infrastructure-with-BitmapHeapSca.patch text/x-patch 21.9 KB
v13-0003-auto_explain-Add-new-GUC-auto_explain.log_io.patch text/x-patch 3.6 KB
v13-0004-Add-EXPLAIN-IO-instrumentation-for-SeqScan.patch text/x-patch 14.7 KB
v13-0005-Add-EXPLAIN-IO-instrumentation-for-TidRangeScan.patch text/x-patch 12.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jelte Fennema-Nio 2026-04-07 12:36:02 Re: pg_get__*_ddl consolidation
Previous Message Alvaro Herrera 2026-04-07 12:33:50 Re: Adding REPACK [concurrently]