Re: pgsql: Make crash recovery ignore recovery target settings. - Mailing list pgsql-committers

From Fujii Masao
Subject Re: pgsql: Make crash recovery ignore recovery target settings.
Date
Msg-id CAHGQGwG_HKncN-EHHPqoWUR6ivuDYjnQthwL=pUKUzyv2VimuA@mail.gmail.com
Whole thread Raw
In response to Re: pgsql: Make crash recovery ignore recovery target settings.  (Michael Paquier <michael@paquier.xyz>)
Responses Re: pgsql: Make crash recovery ignore recovery target settings.  (David Steele <david@pgmasters.net>)
List pgsql-committers
On Mon, Sep 30, 2019 at 11:45 AM Michael Paquier <michael@paquier.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.

Regards,

-- 
Fujii Masao



pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: pgsql: Make crash recovery ignore recovery target settings.
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Add libpq parameter 'channel_binding'.