Re: add PROCESS_MAIN to VACUUM

Nathan Bossart <nathandbossart@gmail.com>

From: Nathan Bossart <nathandbossart@gmail.com>

To: pgsql-hackers@postgresql.org, michael@paquier.xyz

Date: 2023-03-06T22:43:10Z

Lists: pgsql-hackers

Attachments

process_main_readability_fix_v2.patch (text/x-diff) patch v2

On Mon, Mar 06, 2023 at 05:09:58PM -0500, Melanie Plageman wrote:
> I would move this comment inside of the outer if statement since it is
> distinguishing between the two branches of the inner if statement.

Oops, done.

> Also, I would still consider putting a comment above that reminds us that
> VACOPT_PROCESS_MAIN is the default and will vacuum the main relation.

I tried adding something along these lines.

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com

Commits

Improve readability of code PROCESS_MAIN in vacuum_rel()
- ee56048b0ecf 16.0 landed
Add PROCESS_MAIN to VACUUM
- 4211fbd8413b 16.0 landed
Improve the regression tests of VACUUM (PROCESS_TOAST)
- 46d490ac19a7 16.0 landed
Display the leader apply worker's PID for parallel apply workers.
- d540a02a724b 16.0 cited