Re: pgsql: Make crash recovery ignore recovery target settings.

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Fujii Masao <fujii(at)postgresql(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Make crash recovery ignore recovery target settings.
Date: 2019-09-30 03:02:05
Message-ID: CAHGQGwG_HKncN-EHHPqoWUR6ivuDYjnQthwL=pUKUzyv2VimuA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

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.

Regards,

--
Fujii Masao

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2019-09-30 03:39:12 Re: pgsql: Add libpq parameter 'channel_binding'.
Previous Message Michael Paquier 2019-09-30 02:44:54 Re: pgsql: Make crash recovery ignore recovery target settings.