At 2014-06-13 13:39:58 +0200, and...@2ndquadrant.com wrote: > > > void (*rm_desc) (StringInfo buf, uint8 xl_info, char *rec); > > void (*rm_desc) (StringInfo buf, XLogRecord *record); > > > > […] > > +1. I've found this annoying in the past.
I like it too. I was just moving some code from pg_xlogdump into another (new) rm_desc-like callback, and passing in the XLogRecord makes much more sense to me. -- Abhijit -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers