Horton

Re: add PROCESS_MAIN to VACUUM

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: Melanie Plageman <melanieplageman@gmail.com>, pgsql-hackers@postgresql.org
Date: 2023-03-07T03:45:08Z
Lists: pgsql-hackers 
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?
--
Michael

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.