From: | David Steele <david(at)pgmasters(dot)net> |
---|---|
To: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Make crash recovery ignore recovery target settings. |
Date: | 2019-09-30 11:28:27 |
Message-ID: | eb2ffb53-4ba7-acc9-4fa8-9b75bfa0ef40@pgmasters.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 9/29/19 11:02 PM, Fujii Masao wrote:
> On Mon, Sep 30, 2019 at 11:45 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>>
>> Hi Fujii-san,
>>
>> On Mon, Sep 30, 2019 at 01:21:00AM +0000, Fujii Masao wrote:
>>> Make crash recovery ignore recovery target settings.
>>>
>>> In v11 or before, recovery target settings could not take effect in
>>> crash recovery because they are specified in recovery.conf and
>>> crash recovery always starts without recovery.conf. But commit
>>> 2dedf4d9a8 integrated recovery.conf into postgresql.conf and
>>> which unexpectedly allowed recovery target settings to take effect
>>> even in crash recovery. This is definitely not good behavior.
>>>
>>> To fix the issue, this commit makes crash recovery always ignore
>>> recovery target settings.
>>>
>>> Back-patch to v12.
>>
>> It would be nice to avoid committing stuff into REL_12_STABLE until
>> the version is tagged (not stamped!) as the tarball wrap for the GA of
>> this week is going to be soon.
>
> I was thinking this is open item for v12 and needs to be fixed before GA.
> But per discussion in pgsql-release, the consensus seems to be to
> ship GA first and fix that after GA. Sorry for my inattention...
> I suspend to commit other recovery-related things until GA is tagged.
I'm surprised that we would consider going to GA with an issue like this
outstanding.
--
-David
david(at)pgmasters(dot)net
From | Date | Subject | |
---|---|---|---|
Next Message | Fujii Masao | 2019-09-30 13:14:26 | Re: pgsql: doc: Further clarify how recovery target parameters are applied |
Previous Message | Michael Paquier | 2019-09-30 07:08:26 | Re: pgsql: Add libpq parameter 'channel_binding'. |