Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode

Поиск
Список
Период
Сортировка
От Magnus Hagander
Тема Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode
Дата
Msg-id CABUevExvZxy17kk8d9ARd0ZQad-4rBH1_gCXrMBaTELb2D6U8A@mail.gmail.com
обсуждение исходный текст
Ответ на Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-committers


On Sun, Oct 23, 2016 at 5:48 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
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.

Sure, that should work too. Seems we're not defining that one to something else, so let's use that.

I'll go fix. 


--

В списке pgsql-committers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: pgsql: Allow pg_basebackup to stream transaction log in tar mode
Следующее
От: Magnus Hagander
Дата:
Сообщение: pgsql: Rename walmethod fsync method to sync