Re: pg_execute_from_file, patch v10
Itagaki Takahiro <itagaki.takahiro@gmail.com>
From: Itagaki Takahiro <itagaki.takahiro@gmail.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Dimitri Fontaine <dimitri@2ndquadrant.fr>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2010-12-15T22:05:27Z
Lists: pgsql-hackers
Attachments
- pg_read_binary_file.patch (application/octet-stream) patch
On Wed, Dec 15, 2010 at 12:55, Robert Haas <robertmhaas@gmail.com> wrote: >>> It seems like pg_read_binary_file() is good to have regardless of >>> whatever else we decide to do here. Should we pull that part out and >>> commit it separately? >> > The whole-file versions seem like a good idea - my only hesitation is, > I'm not sure why we didn't include that functionality originally. It > seems obviously useful, so does that mean that it was omitted on > purpose for some reason? I applied the attached patch extracted from Dimitri's work. One difference is 'offset' argument is removed from 'whole' mode. So, we'll have (path, offset, length) and (path) versions. Checking with convert_and_check_filename is left as-is. -- Itagaki Takahiro