From: | Andres Freund <andres(at)2ndquadrant(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Cc: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [PATCH 02/14] Add support for a generic wal reading facility dubbed XLogReader |
Date: | 2012-11-15 15:12:14 |
Message-ID: | 20121115151214.GB26088@awork2.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2012-11-15 11:50:37 -0300, Alvaro Herrera wrote:
> Heikki Linnakangas wrote:
> > On 15.11.2012 03:17, Andres Freund wrote:
> > >
> > >Features:
> > >- streaming reading/writing
> > >- filtering
> > >- reassembly of records
> > >
> > >Reusing the ReadRecord infrastructure in situations where the code that wants
> > >to do so is not tightly integrated into xlog.c is rather hard and would require
> > >changes to rather integral parts of the recovery code which doesn't seem to be
> > >a good idea.
> > >
> > >Missing:
> > >- "compressing" the stream when removing uninteresting records
> > >- writing out correct CRCs
> > >- separating reader/writer
> >
> > I'm disappointed to see that there has been no progress on this
> > patch since last commitfest. I thought we agreed on the approach I
> > championed for here:
> > http://archives.postgresql.org/pgsql-hackers/2012-09/msg00636.php.
> > There wasn't much work left to finish that, I believe.
> >
> > Are you going to continue working on this?
>
> I worked a bit more on that patch of yours, but I neglected to submit
> it. Did you have something in particular that you wanted changed in it?
Could you push your newest version to your git repository or similar?
Greetings,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2012-11-15 15:16:20 | Re: Switching timeline over streaming replication |
Previous Message | Robert Haas | 2012-11-15 15:11:58 | Re: [PATCH] binary heap implementation |