Re: BUG #6372: Error while creating database with fsync parameter as on incase of CIFS

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, anjali_524 <anjali_524(at)yahoo(dot)co(dot)in>, Pg Bugs <pgsql-bugs(at)postgresql(dot)org>
Subject: Re: BUG #6372: Error while creating database with fsync parameter as on incase of CIFS
Date: 2012-01-02 21:00:19
Message-ID: 2113.1325538019@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> Excerpts from Tom Lane's message of lun ene 02 17:28:33 -0300 2012:
>> it seems like EINVAL is a considerably more reasonable thing to return
>> than EBADF, if the filesystem is trying to tell you that it won't fsync
>> a directory. So I'm a bit surprised this question hasn't come up for
>> other filesystems.

> Probably because other filesystems do allow you to fsync directories.
> In fact for some cases they _require_ it ... remember the fiasco when
> MTA writers were told that they needed to fsync their queue dirs in
> order for all queued email to persist?

Yeah, the long and the short of it is that if the filesystem won't
accept an fsync on a directory, we have to assume that it doesn't need
it and will manage metadata persistence safely without prodding.

The only real question here is whether an EINVAL could mean something
besides "fsync on directory is not accepted". If there are any
scenarios where it represents a transient/fixable error, then we'd
want to report it. It's far from clear to me that there are any
though. What it could mean in general is not at issue, because we
know the target is a directory that we just created moments before.

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message valgog 2012-01-03 18:04:23 BUG #6375: tsearch does not recognize all valid emails
Previous Message Alvaro Herrera 2012-01-02 20:53:08 Re: BUG #6372: Error while creating database with fsync parameter as on incase of CIFS