Re: WIP patch: add (PRE|POST)PROCESSOR options to COPY

Etsuro Fujita <fujita.etsuro@lab.ntt.co.jp>

From: "Etsuro Fujita" <fujita.etsuro@lab.ntt.co.jp>
To: "'Tom Lane'" <tgl@sss.pgh.pa.us>
Cc: "'Craig Ringer'" <ringerc@ringerc.id.au>, <pgsql-hackers@postgresql.org>
Date: 2012-09-19T06:05:55Z
Lists: pgsql-hackers
> From: Tom Lane [mailto:tgl@sss.pgh.pa.us]

> I wrote:
> > "Etsuro Fujita" <fujita.etsuro@lab.ntt.co.jp> writes:
> >> I have a question.  I think it would be also better to extend the syntax
> >> for the SQL COPY command in the same way, ie,
> >> COPY foo from '/home/pgsql/decompress.sh /home/pgsql/foo.csv.gz |' with
> >> format 'csv'
> 
> > Yeah, sure --- that case is already superuser-only, so why not give it
> > the option of being a popen instead of just fopen.
> 
> BTW, one thought that comes to mind is that such an operation is
> extremely likely to fail under environments such as SELinux.  That's
> not necessarily a reason not to do it, but we should be wary of
> promising that it will work everywhere.  Probably a documentation note
> about this would be enough.

OK  I'll revise the patch.

Thank you for your advice!

Best regards,
Etsuro Fujita