Horton

Re: add PROCESS_MAIN to VACUUM

Melanie Plageman <melanieplageman@gmail.com>

View thread View on postgr.es
From: Melanie Plageman <melanieplageman@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Nathan Bossart <nathandbossart@gmail.com>, pgsql-hackers@postgresql.org
Date: 2023-03-07T17:39:29Z
Lists: pgsql-hackers 
On Mon, Mar 6, 2023 at 10:45 PM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Mon, Mar 06, 2023 at 04:59:49PM -0800, Nathan Bossart wrote:
> > That did cross my mind, but I was worried that trying to explain all that
> > here could cause confusion.
> >
> >       If PROCESS_MAIN is set (the default), it's time to vacuum the main
> >       relation.  Otherwise, we can skip this part.  If processing the TOAST
> >       table is required (e.g., PROCESS_TOAST is set), we'll force
> >       PROCESS_MAIN to be set when we recurse to the TOAST table so that it
> >       gets processed here.
> >
> > How does that sound?
>
> Sounds clear to me, thanks!  Melanie, what do you think?

Yes, sounds clear to me also!

- Melanie

Commits

  1. Improve readability of code PROCESS_MAIN in vacuum_rel()

  2. Add PROCESS_MAIN to VACUUM

  3. Improve the regression tests of VACUUM (PROCESS_TOAST)

  4. Display the leader apply worker's PID for parallel apply workers.