Re: WIP patch: add (PRE|POST)PROCESSOR options to COPY - Mailing list pgsql-hackers

From Etsuro Fujita
Subject Re: WIP patch: add (PRE|POST)PROCESSOR options to COPY
Date
Msg-id 008801cdc8aa$5062cc20$f1286460$@lab.ntt.co.jp
Whole thread Raw
In response to Re: WIP patch: add (PRE|POST)PROCESSOR options to COPY  ("Etsuro Fujita" <fujita.etsuro@lab.ntt.co.jp>)
Responses Re: WIP patch: add (PRE|POST)PROCESSOR options to COPY  ("Etsuro Fujita" <fujita.etsuro@lab.ntt.co.jp>)
List pgsql-hackers
I wrote:
> > The biggest problem this patch has had from the very beginning is
> > overdesign, and this is more of the same.  Let's please just define the
> > feature as "popen, not fopen, the given string" and have done.  You can
> > put all the warning verbiage you want in the documentation.  (But note
> > that the server-side version would be superuser-only in any flavor of
> > the feature.)
> 
> Agreed.  I'll reimplement the feature using the PROGRAM keyword:
> 
> >     COPY TABLE FROM PROGRAM 'command line';
> >     COPY TABLE TO PROGRAM 'command line';

I've reimplemented the feature.  Attached is an updated version of the patch.

Todo:   * More documents   * More tests

Any comments are welcomed.

Thanks,

Best regards,
Etsuro Fujita

pgsql-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: PQconninfo function for libpq
Next
From: pl65
Date:
Subject: Re: C-function, don't load external dll file