Re: [COMMITTERS] pgsql: Convert wal_sync_method to guc enum. - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: [COMMITTERS] pgsql: Convert wal_sync_method to guc enum.
Date
Msg-id 20080512171028.4573e930@mha-laptop.hagander.net
Whole thread Raw
Responses Re: [COMMITTERS] pgsql: Convert wal_sync_method to guc enum.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
Magnus Hagander wrote:
> Tom Lane wrote:
> > mha@postgresql.org (Magnus Hagander) writes:
> > > Convert wal_sync_method to guc enum.
> > 
> > Buildfarm says you broke things for Windows.
> 
> Yeah, working on that with Dave. First part was to unbreak the error
> message so we can actually figure out what's broken :-(
> 

I need to leave for a couple of hours, will look again when I get back.
But so far, I'm quite surprised. Here's my reasoning, please poke holes
in it :-)

1) Win32 always defines O_DSYNC (win32.h)
2) That means we should always define OPEN_DATASYNC_FLAG (xlogdefs.h,
line 107)
3) That means that the error should not happen at all, because of
xlog.c line 6358.

Anybody who can kill this argument before I get back ;-) It's obviously
flawed somewhere...

//Magnus


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [0/4] Proposal of SE-PostgreSQL patches
Next
From: Alvaro Herrera
Date:
Subject: Re: bloated heapam.h