| 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 |
| 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] |