From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Magnus Hagander <magnus(at)hagander(dot)net>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Recovery target 'immediate' |
Date: | 2013-05-03 00:54:15 |
Message-ID: | CAB7nPqSU6O+COCq5jSMdKfddNHfWtKmH0_3RkRNX6Ng2Y4gRUg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, May 3, 2013 at 8:56 AM, Bruce Momjian <bruce(at)momjian(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2013-05-03 01:04:04 | Re: \watch stuck on execution of commands returning no tuples |
Previous Message | Bruce Momjian | 2013-05-03 00:12:32 | Re: Remaining beta blockers |