From: | Andres Freund <andres(at)2ndquadrant(dot)com> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | pgsql-hackers(at)postgresql(dot)org, Simon Riggs <simon(at)2ndquadrant(dot)com>, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
Subject: | Re: XLog changes for 9.3 |
Date: | 2012-06-07 16:02:56 |
Message-ID: | 201206071802.56490.andres@2ndquadrant.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thursday, June 07, 2012 06:02:12 PM Magnus Hagander wrote:
> On Thu, Jun 7, 2012 at 5:56 PM, Andres Freund <andres(at)2ndquadrant(dot)com>
wrote:
> > Hi,
> >
> > On Thursday, June 07, 2012 05:51:07 PM Simon Riggs wrote:
> >> On 7 June 2012 14:50, Heikki Linnakangas
> >>
> >> <heikki(dot)linnakangas(at)enterprisedb(dot)com> wrote:
> >> > These changes will help the XLogInsert scaling patch
> >>
> >> ...and as I'm sure you're aware will junk much of the replication code
> >> and almost certainly set back the other work that we have brewing for
> >> 9.3. So this is a very large curve ball you're throwing there.
> >
> > It's not that bad. Most of that code is pretty abstracted, the changes to
> > adapt to that should be less than 20 lines. And it would remove some of
> > the complexity.
> >
> >> Personally, I don't think we should do this until we have a better
> >> regression test suite around replication and recovery because the
> >> impact will be huge but I welcome the suggested changes themselves.
> >
> > Hm. One could regard the logical rep stuff as a testsuite ;)
> >
> >> If you are going to do this in 9.3, then it has to be early in the
> >> first Commit Fest and you'll need to be around to quickly follow
> >> through on all of the other subsequent breakages it will cause,
> >> otherwise every other piece of work in this area will be halted or
> >> delayed.
> >
> > Yea, I would definitely welcome an early patch.
>
> Just as I'm sure everybody else would welcome *your* patches landing
> in the first commitfest and that you all guarantee to be around
> quickly follow through on all potential breakages *that* can cause.
Agreed.
Andres
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2012-06-07 16:08:34 | Re: "page is not marked all-visible" warning in regression tests |
Previous Message | Magnus Hagander | 2012-06-07 16:02:12 | Re: XLog changes for 9.3 |