From: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
---|---|
To: | Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp> |
Cc: | Nathan Bossart <nathandbossart(at)gmail(dot)com>, Corey Huinker <corey(dot)huinker(at)gmail(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Don't treat virtual generated columns as missing statistics in vacuumdb --missing-stats-only |
Date: | 2025-08-22 01:19:32 |
Message-ID: | CAHGQGwGsErH4mXcdtGZVvT82Pk6S7cmjNh77-hrdh3iLAfXZ7g@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Aug 22, 2025 at 2:23 AM Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp> wrote:
>
> On Thu, 21 Aug 2025 10:37:10 -0500
> Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>
> > On Thu, Aug 21, 2025 at 11:29:56AM -0400, Corey Huinker wrote:
> > > On Thu, Aug 21, 2025 at 9:56 AM Nathan Bossart <nathandbossart(at)gmail(dot)com>
> > > wrote:
> > >> Since we're running out of time for v18, I went ahead and updated the
> > >> patch. I've also added an open item for this.
> > >
> > > +1 on the fix, works for me.
> >
> > Thanks. I'll plan on committing this in roughly 24 hours (barring
> > additional feedback).
Thanks a lot!
> Thank you for updating the patch.
> I'm fine with your fixes, both in the test and in the additional change
> for inheritance tables. I had overlooked the latter one.
The patch looks good to me, too.
At first, I just wondered whether vacuumdb --missing-stats-only would work
on older servers, since the patch adds access to the attgenerated column,
which might not exist there. But that's not an issue, because
--missing-stats-only is supported only on servers version 15 or later,
where attgenerated is available.
Regards,
--
Fujii Masao
From | Date | Subject | |
---|---|---|---|
Next Message | Japin Li | 2025-08-22 01:29:20 | Re: Standardize LSN-based filename |
Previous Message | Chao Li | 2025-08-22 00:44:17 | Re: Remove unneeded cast in heap_xlog_lock. |