Re: Recovery target 'immediate' - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Recovery target 'immediate'
Date
Msg-id CAB7nPqSU6O+COCq5jSMdKfddNHfWtKmH0_3RkRNX6Ng2Y4gRUg@mail.gmail.com
Whole thread
In response to Re: Recovery target 'immediate'  (Bruce Momjian <bruce@momjian.us>)
Responses Re: Recovery target 'immediate'
List pgsql-hackers



On Fri, May 3, 2013 at 8:56 AM, Bruce Momjian <bruce@momjian.us> wrote:
On Thu, May  2, 2013 at 09:31:03AM +0200, Magnus Hagander wrote:
> Actually, there is - I hear it quite often from people not so
> experienced in PostgreSQL. Though in fairness, I'm not entirely sure
> the new syntax would help - some of those need a tool to do it for
> them, really (and such tools exist, I believe).
>
> That said, there is one property that's very unclear now and that's
> that you can only set one of recovery_target_time, recovery_target_xid
> and recovery_target_name. But they can be freely combined with
> recovery_target_timeline and recovery_target_inclusive. That's quite
> confusing.
>
>
>
> > This changes the existing API which will confuse people that know it
> > and invalidate everything written in software and on wikis as to how
> > to do it. That means all the "in case of fire break glass"
> > instructions are all wrong and need to be rewritten and retested.
>
> Yes, *that* is the main reason *not* to make the change. It has a
> pretty bad cost in backwards compatibility loss. There is a gain, but
> I don't think it outweighs the cost.

So, is there a way to add this feature without breaking the API?
Yes, by adding a new parameter exclusively used to control this feature, something like recovery_target_immediate = 'on/off'.
--
Michael

pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Remaining beta blockers
Next
From: Michael Paquier
Date:
Subject: Re: \watch stuck on execution of commands returning no tuples