Re: add PROCESS_MAIN to VACUUM

From: Melanie Plageman <melanieplageman(at)gmail(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org, michael(at)paquier(dot)xyz
Subject: Re: add PROCESS_MAIN to VACUUM
Date: 2023-03-06 23:12:36
Message-ID: CAAKRu_ZEKEsXDfFbOHRMhEROK5h1KcTcwUhERv5ZE0RQjX82_A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Mar 6, 2023 at 5:43 PM Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>
> 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.

LGTM.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Regina Obe 2023-03-06 23:26:59 RE: Ability to reference other extensions by schema in extension scripts
Previous Message Nathan Bossart 2023-03-06 22:43:10 Re: add PROCESS_MAIN to VACUUM