Re: pgsql: Move the backup-block logic from XLogInsert to a new file, xlogi - Mailing list pgsql-committers

From Heikki Linnakangas
Subject Re: pgsql: Move the backup-block logic from XLogInsert to a new file, xlogi
Date
Msg-id 545D3537.8080708@vmware.com
Whole thread Raw
In response to Re: pgsql: Move the backup-block logic from XLogInsert to a new file, xlogi  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-committers
On 11/07/2014 08:28 PM, Alvaro Herrera wrote:
> Heikki Linnakangas wrote:
>> Move the backup-block logic from XLogInsert to a new file, xloginsert.c.
>>
>> xlog.c is huge, this makes it a little bit smaller, which is nice. Functions
>> related to putting together the WAL record are in xloginsert.c, and the
>> lower level stuff for managing WAL buffers and such are in xlog.c.
>>
>> Also move the definition of XLogRecord to a separate header file. This
>> causes churn in the #includes of all the files that write WAL records, and
>> redo routines, but it avoids pulling in xlog.h into most places.
>
> This commit broke the WAL_DEBUG case:
>
> /pgsql/source/brin/src/backend/access/transam/xlog.c: In function 'XLogInsertRecord':
> /pgsql/source/brin/src/backend/access/transam/xlog.c:1073:4: error: 'rdt_lastnormal' undeclared (first use in this
function)
>      rdt_lastnormal->next = NULL;
>      ^

Sorry, fixed.

- Heikki



pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Fix building with WAL_DEBUG.
Next
From: Robert Haas
Date:
Subject: pgsql: Introduce custom path and scan providers.