From: Justin Pryzby <pryzby@telsasoft.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Bruce Momjian <bruce@momjian.us>
Cc: pgsql-hackers@postgresql.org
Subject: Re: PG 14 release notes, first draft
Date: Sat, 18 Sep 2021 16:35:51 -0500
Message-ID: <20210918213551.GV831@telsasoft.com> (raw)
In-Reply-To: <1500489.1631999739@sss.pgh.pa.us>
References: <20210918204605.GU831@telsasoft.com>
<1500489.1631999739@sss.pgh.pa.us>
On Sat, Sep 18, 2021 at 05:15:39PM -0400, Tom Lane wrote:
> Justin Pryzby <pryzby@telsasoft.com> writes:
> > I think the release notes for the autovacuum item (which was first reverted and
> > then partially un-reverted) should say something like "Partitioned tables are
> > now included in pg_stat_all_tables":
> > | e1efc5b465 Keep stats up to date for partitioned tables
>
> Hmm. If I'm reading the commit message properly, the actual change there
> is not that, but that analyze count and last analyze time are now tracked
> correctly for partitioned tables. Might be worth mentioning, not sure.
The reverted patch to autoanalyze included partitioned tables in
pg_stat_all_tables, and the revert specifically avoided changing that (to avoid
a catbump). But last_analyzed and analyze_count were always shown as "0". So
the e1 commit addresses that by tracking that information and showing correct
value instead of always 0.
The relevant portion starts here:
https://www.postgresql.org/message-id/flat/202108161700.d4eh6a7n2lki%40alvherre.pgsql#b2e426eb19dbbd...
I suggest that this *should* be included in the release notes, since I
specifically requested that partitioned tables be included in 2018.
> In 20200418050815(dot)GE26953(at)telsasoft(dot)com I wrote:
> |This patch includes partitioned tables in pg_stat_*_tables, which is great; I
> |complained awhile ago that they were missing [0]. It might be useful if that
> |part was split out into a separate 0001 patch (?).
> | [0] https://www.postgresql.org/message-id/20180601221428.GU5164%40telsasoft.com
Also, I've patched my analyze script to use that field (same as for
nonpartitioned tables) rather than needing to do a subquery involving
max(last_analyzed) of the partitions. Since it's still needed to manually
analyze parent tables.
--
Justin
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-hackers@postgresql.org
Cc: pryzby@telsasoft.com, tgl@sss.pgh.pa.us, bruce@momjian.us
Subject: Re: PG 14 release notes, first draft
In-Reply-To: <20210918213551.GV831@telsasoft.com>
* 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