From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Peter Geoghegan <pg(at)bowt(dot)ie> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Showing primitive index scan count in EXPLAIN ANALYZE (for skip scan and SAOP scans) |
Date: | 2024-08-27 19:04:15 |
Message-ID: | 3124106.1724785455@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Peter Geoghegan <pg(at)bowt(dot)ie> writes:
> I see value in making it obvious to users when and how
> pg_stat_all_indexes.idx_scan advances. Being able to easily relate it
> to EXPLAIN ANALYZE output is useful, independent of whether or not
> SAOPs happen to be used. That's probably the single best argument in
> favor of showing "Index Searches: N" unconditionally. But I'm
> certainly not going to refuse to budge over that.
TBH, I'm afraid that this patch basically is exposing numbers that
nobody but Peter Geoghegan and maybe two or three other hackers
will understand, and even fewer people will find useful (since the
how-many-primitive-scans behavior is not something users have any
control over, IIUC). I doubt that "it lines up with
pg_stat_all_indexes.idx_scan" is enough to justify the additional
clutter in EXPLAIN. Maybe we should be going the other direction
and trying to make pg_stat_all_indexes count in a less detailed but
less surprising way, ie once per indexscan plan node invocation.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2024-08-27 19:11:15 | Re: allowing extensions to control planner behavior |
Previous Message | Nathan Bossart | 2024-08-27 18:55:35 | Re: remove adaptive spins_per_delay code |