Re: [PATCH] XLogReader v2 - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [PATCH] XLogReader v2
Date
Msg-id CA+TgmobO4ABZNSt=9yKOsZRvcA=_yomvhOm+UekK6Oick+8dJg@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] XLogReader v2  (Andres Freund <andres@2ndquadrant.com>)
Responses Re: [PATCH] XLogReader v2
List pgsql-hackers
On Mon, Jul 23, 2012 at 12:13 PM, Andres Freund <andres@2ndquadrant.com> wrote:
>> Could that be fixed by moving the debugging routines into a separate
>> set of files, instead of having them lumped in with the code that
>> applies those xlog records?
> Its a major effort. Those function use elog(), stringinfo and lots of other
> stuff... I am hesitant to start working on that.
> On the other hand - I think an in-core xlogdump would be great and sensible
> thing; but I can live with using my hacked up version that simply links to the
> backend...

The stringinfo thing has long been an annoyance to me.  libpq has
PQExpBuffer which is the exact same thing.  I don't like that we have
two implementations of that in two different code bases, and you have
to remember to spell it right depending on where you are.  I'm not
sure exactly what the best way to fix that is, but it sure is a pain
in the neck.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: [PATCH] XLogReader v2
Next
From: Satoshi Nagayasu
Date:
Subject: Re: [PATCH] XLogReader v2