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. +
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 |