Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode
Date
Msg-id 26273.1477237689@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode  (Magnus Hagander <magnus@hagander.net>)
Responses Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode
List pgsql-committers
Magnus Hagander <magnus@hagander.net> writes:
> The remaining windows buildfarm build issue looks like this:
>   src/bin/pg_basebackup/receivelog.c(135): error C2039: '_commit' : is
> not a member of 'WalWriteMethod'
> [C:\buildfarm\buildenv\HEAD\pgsql.build\pg_basebackup.vcxproj]
> AFAICT this comes from using walmethod->fsync(), and we have a #define
> changing fsync() to _commit (port/win32.h, line 70).

Yeah.

> The easiest might just be to rename walmethod->fsync() to
> walmethod->do_fsync(), so we don't have to mess with any other code.

How about calling the method just "sync()"?  Seems a bit less
artificial.

            regards, tom lane


pgsql-committers by date:

Previous
From: Magnus Hagander
Date:
Subject: pgsql: Fix obviously too quickly applied fix to zlib issue
Next
From: Magnus Hagander
Date:
Subject: Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode