From: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Reviewing freeze map code |
Date: | 2016-06-06 10:22:34 |
Message-ID: | CAD21AoADsfy6bys0LFCNKONjb0+1W4mEzxsVBbb6jc3VpNPwYw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Jun 6, 2016 at 6:34 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Mon, Jun 6, 2016 at 5:11 AM, Michael Paquier
> <michael(dot)paquier(at)gmail(dot)com> wrote:
>>> Attached is a sample patch that controls full page vacuum by new GUC parameter.
>>
>> Don't we want a reloption for that? Just wondering...
>
> Why? Just for consistency? I think the bigger question here is
> whether we need to do anything at all. It's true that, without some
> new option, we'll lose the ability to forcibly vacuum every page in
> the relation, even if all-frozen. But there's not much use case for
> that in the first place. It will be potentially helpful if it turns
> out that we have a bug that sets the all-frozen bit on pages that are
> not, in fact, all-frozen. Otherwise, what's the use?
>
I cannot agree with using this parameter as a reloption.
We set it true only when the serious bug is discovered and we want to
re-generate the visibility maps of specific tables.
I thought that control by GUC parameter would be convenient rather
than adding the new option.
Regards,
--
Masahiko Sawada
From | Date | Subject | |
---|---|---|---|
Next Message | Sridhar N Bamandlapally | 2016-06-06 11:17:15 | OUT parameter and RETURN table/setof |
Previous Message | Amit Kapila | 2016-06-06 10:07:33 | Re: [sqlsmith] Failed assertion in parallel worker (ExecInitSubPlan) |