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
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. |