Re: pgsql: Add missing_ok option to the SQL functions for reading files.

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: Heikki Linnakangas <heikki(dot)linnakangas(at)iki(dot)fi>, pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Add missing_ok option to the SQL functions for reading files.
Date: 2015-08-28 12:39:26
Message-ID: 20150828123926.GA21830@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Sun, Jul 5, 2015 at 08:44:41PM +0900, Michael Paquier wrote:
> On Mon, Jun 29, 2015 at 3:39 AM, Heikki Linnakangas
> <heikki(dot)linnakangas(at)iki(dot)fi> wrote:
> > Add missing_ok option to the SQL functions for reading files.
> >
> > This makes it possible to use the functions without getting errors, if there
> > is a chance that the file might be removed or renamed concurrently.
> > pg_rewind needs to do just that, although this could be useful for other
> > purposes too. (The changes to pg_rewind to use these functions will come in
> > a separate commit.)
> >
> > The read_binary_file() function isn't very well-suited for extensions.c's
> > purposes anymore, if it ever was. So bite the bullet and make a copy of it
> > in extension.c, tailored for that use case. This seems better than the
> > accidental code reuse, even if it's a some more lines of code.
>
> This has been done done as the part of a fix for the issues with
> pg_rewind, but perhaps it should be mentioned in the release notes?

Yes, I will pick up that change during the creation of the 9.6 release
notes.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ Everyone has their own god. +

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2015-08-28 12:41:08 Re: pgsql: Add missing_ok option to the SQL functions for reading files.
Previous Message Peter Eisentraut 2015-08-28 02:50:24 pgsql: Simplify Perl chmod calls