From: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
---|---|
To: | Andres Freund <andres(at)2ndquadrant(dot)com> |
Cc: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: rm_desc signature |
Date: | 2014-06-13 12:30:32 |
Message-ID: | CAHGQGwFSMEZW2cX1GBxxzm7EP-kOgH07H_+e=M7EOCWVg5g3-Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Jun 13, 2014 at 8:39 PM, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
> On 2014-06-13 14:37:33 +0300, Heikki Linnakangas wrote:
>> As part of the WAL-format changing patch I've been working on, I changed the
>> signature of the rm_desc function from:
>>
>> void (*rm_desc) (StringInfo buf, uint8 xl_info, char *rec);
>> void (*rm_desc) (StringInfo buf, XLogRecord *record);
>>
>> The WAL-format patch needed that because it added more functions/macros for
>> working with XLogRecords, also used by rm_desc routines, but it seems like a
>> sensible change anyway. IMHO it was always a bit strange that rm_desc was
>> passed the info field and record payload separately.
+1, too.
-/* #define WAL_DEBUG */
+#define WAL_DEBUG
ISTM you just forgot to exclude this change from the patch.
Regards,
--
Fujii Masao
From | Date | Subject | |
---|---|---|---|
Next Message | Abhijit Menon-Sen | 2014-06-13 12:33:27 | Re: rm_desc signature |
Previous Message | Gurjeet Singh | 2014-06-13 12:07:39 | Re: Proposing pg_hibernate |