MIME-Version: 1.0
References: <E1tp8XG-000baq-1b@gemulon.postgresql.org>
 <202503031644.nihftaytey2b@alvherre.pgsql>
 <CAAKRu_Z_W+Fc92onBM7LmeFs0+Dcu4A7K_e5m6LMMn9biV+_xw@mail.gmail.com>
In-Reply-To: 
 <CAAKRu_Z_W+Fc92onBM7LmeFs0+Dcu4A7K_e5m6LMMn9biV+_xw@mail.gmail.com>
From: Melanie Plageman <melanieplageman@gmail.com>
Date: Mon, 3 Mar 2025 11:52:40 -0500
Message-ID: 
 <CAAKRu_b9MxbXbPFLShzRxgjV3kjxCZ-GrJHVBDvioa37bTgiow@mail.gmail.com>
Subject: Re: pgsql: Add relallfrozen to pg_class
To: =?UTF-8?Q?=C3=81lvaro_Herrera?= <alvherre@alvh.no-ip.org>
Cc: pgsql-committers@lists.postgresql.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAAKRu_b9MxbXbPFLShzRxgjV3kjxCZ-GrJHVBDvioa37bTgiow%40mail.gmail.com>
Precedence: bulk

On Mon, Mar 3, 2025 at 11:48=E2=80=AFAM Melanie Plageman
<melanieplageman@gmail.com> wrote:
>
> On Mon, Mar 3, 2025 at 11:44=E2=80=AFAM =C3=81lvaro Herrera <alvherre@alv=
h.no-ip.org> wrote:
> >
> > I'm confused about this.  Why was the new value added to pg_class
> > instead of to the pgstat system?  I don't think relallvisible is a good
> > precedent, because as you write here, that one is used for planning,
> > which has different requirements.  For vacuuming metrics we rely on
> > pgstat.
>
> We use relpages and reltuples from pg_class in
> relation_needs_vacanalyze() in the same way relallfrozen is being used
> here.

Sorry reltuples only (not relpages).

- Melanie