Re: BUG #13770: Extending recovery_min_apply_delay on Standby causes it to be unavailable for a while - Mailing list pgsql-bugs

From Michael Paquier
Subject Re: BUG #13770: Extending recovery_min_apply_delay on Standby causes it to be unavailable for a while
Date
Msg-id CAB7nPqRnFijSmygzORtfH1t4gn_wU6E6Ng4bT36ENkBSDcUJew@mail.gmail.com
Whole thread Raw
In response to Re: BUG #13770: Extending recovery_min_apply_delay on Standby causes it to be unavailable for a while  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: BUG #13770: Extending recovery_min_apply_delay on Standby causes it to be unavailable for a while  (Michael Paquier <michael.paquier@gmail.com>)
List pgsql-bugs
On Thu, Dec 31, 2015 at 12:35 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Andres Freund <andres@anarazel.de> writes:
>> On 2015-12-26 22:45:57 +0900, Michael Paquier wrote:
>>> Depending on the use cases, it may be interesting to have a switch
>>> allowing to not apply the delay should a consistent point not be
>>> reached though...
>
>> Is there actually any case where it's interesting to delay in that
>> scenario? I mean that really can only happen if you changed the
>> configuration to a different delay, or your clock offset
>> changed. Otherwise we should always reach the consistent point before
>> the delay plays a role.  I'm tempted to simply only check for delay when
>> consistent.
>
> The argument for having a delay at all is to allow backing up to some
> earlier point in the master's history; but a slave that is not yet
> consistent cannot provide any rollback/recovery option.  The slave is
> completely useless for any purpose until it reaches consistency, so
> it might as well do that as fast as possible, and then sit on the
> next WAL record until the delay is met.  +1 for no delay at all when
> not consistent.

OK, I don't mind doing so if you guys think that's more adapted. Based
on reading the code, it seems obvious though that this was made so as
a delay is taken into account even before the node is consistent.
Hence ISTM that it would be good as well to get feedback from people
who actually worked on the feature before deciding anything. Robert,
Simon?
--
Michael

pgsql-bugs by date:

Previous
From: Tom Lane
Date:
Subject: Re: BUG #13770: Extending recovery_min_apply_delay on Standby causes it to be unavailable for a while
Next
From: Bruce Momjian
Date:
Subject: Re: BUG #13783: 'create database test owner testowner' as 'postgres' leaves test.public owned by postgres