Re: Add pg_file_sync() to adminpack - Mailing list pgsql-hackers

From Julien Rouhaud
Subject Re: Add pg_file_sync() to adminpack
Date
Msg-id CAOBaU_YVsOnVUxWhhmSOpPbW_r2_WY8DZdoGY7OHsgPcWULyNw@mail.gmail.com
Whole thread Raw
In response to Add pg_file_sync() to adminpack  (Fujii Masao <masao.fujii@gmail.com>)
Responses Re: Add pg_file_sync() to adminpack
Re: Add pg_file_sync() to adminpack
List pgsql-hackers
On Wed, Dec 25, 2019 at 2:01 PM Fujii Masao <masao.fujii@gmail.com> wrote:
>
> Hi,
>
> I'd like to propose to add pg_file_sync() function into contrib/adminpack.
> This function fsyncs the specified file or directory named by its argument.
> IMO this is useful, for example, when you want to fsync the file that
> pg_file_write() writes out or that COPY TO exports the data into,
> for durability. Thought?

+1, that seems like a useful wrapper.  Looking at existing functions,
I see that there's a pg_file_rename() in adminpack, but it doesn't use
durable_rename nor does it try to perform any fsync.  Same for
pg_file_unlink vs. durable_unlink.  It's probably worth fixing that at
the same time?



pgsql-hackers by date:

Previous
From: Alexey Kondratov
Date:
Subject: Re: Physical replication slot advance is not persistent
Next
From: Maksim Milyutin
Date:
Subject: Re: Disallow cancellation of waiting for synchronous replication