Re: [HACKERS] [PATCH 4/5] Add pg_xlogdump contrib module

2013-02-04 Thread Alvaro Herrera
Andres Freund wrote: > On 2013-02-04 14:55:56 -0300, Alvaro Herrera wrote: > > Andres Freund wrote: > > > On 2013-02-04 13:22:26 -0300, Alvaro Herrera wrote: > > > > > > +typedef struct RmgrDescData > > > > +{ > > > > + void(*rm_desc) (StringInfo buf, uint8 xl_info, char > > > >

Re: [HACKERS] [PATCH 4/5] Add pg_xlogdump contrib module

2013-02-04 Thread Andres Freund
On 2013-02-04 14:55:56 -0300, Alvaro Herrera wrote: > Andres Freund wrote: > > On 2013-02-04 13:22:26 -0300, Alvaro Herrera wrote: > > > > +typedef struct RmgrDescData > > > +{ > > > + void(*rm_desc) (StringInfo buf, uint8 xl_info, char *rec); > > > +} RmgrDescData; > > > + > > > +exte

Re: [HACKERS] [PATCH 4/5] Add pg_xlogdump contrib module

2013-02-04 Thread Alvaro Herrera
Andres Freund wrote: > On 2013-02-04 13:22:26 -0300, Alvaro Herrera wrote: > > +typedef struct RmgrDescData > > +{ > > + void(*rm_desc) (StringInfo buf, uint8 xl_info, char *rec); > > +} RmgrDescData; > > + > > +extern const RmgrDescData RmgrDescTable[]; > > + > > I think we would a

Re: [HACKERS] [PATCH 4/5] Add pg_xlogdump contrib module

2013-02-04 Thread Andres Freund
On 2013-02-04 13:22:26 -0300, Alvaro Herrera wrote: > > I didn't like this bit too much: > > > diff --git a/contrib/pg_xlogdump/tables.c b/contrib/pg_xlogdump/tables.c > > new file mode 100644 > > index 000..e947e0d > > --- /dev/null > > +++ b/contrib/pg_xlogdump/tables.c > > @@ -0,0 +1,78 @@

Re: [HACKERS] [PATCH 4/5] Add pg_xlogdump contrib module

2013-02-04 Thread Alvaro Herrera
I didn't like this bit too much: > diff --git a/contrib/pg_xlogdump/tables.c b/contrib/pg_xlogdump/tables.c > new file mode 100644 > index 000..e947e0d > --- /dev/null > +++ b/contrib/pg_xlogdump/tables.c > @@ -0,0 +1,78 @@ > +/* > + * RmgrTable linked only to functions available outside of

[HACKERS] [PATCH 4/5] Add pg_xlogdump contrib module

2013-01-08 Thread Andres Freund
From: Andres Freund Authors: Andres Freund, Heikki Linnakangas --- contrib/Makefile | 1 + contrib/pg_xlogdump/Makefile | 37 +++ contrib/pg_xlogdump/compat.c | 58 contrib/pg_xlogdump/pg_xlogdump.c | 654 ++ contrib/pg_xlog