Re: Add MAIN_RELATION_CLEANUP and SECONDARY_RELATION_CLEANUP options to VACUUM

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: "Bossart, Nathan" <bossartn(at)amazon(dot)com>
Cc: Justin Pryzby <pryzby(at)telsasoft(dot)com>, Vik Fearing <vik(dot)fearing(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, "k(dot)jamison(at)fujitsu(dot)com" <k(dot)jamison(at)fujitsu(dot)com>
Subject: Re: Add MAIN_RELATION_CLEANUP and SECONDARY_RELATION_CLEANUP options to VACUUM
Date: 2021-02-08 07:35:19
Message-ID: YCDpt53+QCZ+CqEo@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Jan 29, 2021 at 06:43:44PM +0000, Bossart, Nathan wrote:
> I changed it to PROCESS_TOAST.

Thanks. PROCESS_TOAST sounds good to me at the end for the option
name, so let's just go with that.

> Done.

While on it, I could not resist with changing VACOPT_SKIPTOAST to
VACOPT_PROCESS_TOAST on consistency grounds. This is used only in
four places in the code, so that's not invasive.

What do you think?
--
Michael

Attachment Content-Type Size
v8-0001-Add-PROCESS_TOAST-option-to-VACUUM.patch text/x-diff 13.1 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andy Fan 2021-02-08 07:43:47 Re: Extend more usecase for planning time partition pruning and init partition pruning.
Previous Message Greg Nancarrow 2021-02-08 07:14:42 Re: Parallel INSERT (INTO ... SELECT ...)