* Tom Lane (tgl(at)sss(dot)pgh(dot)pa(dot)us) wrote: > Andrew Dunstan <andrew(at)dunslane(dot)net> writes: > > ... Oh, and if we did allow the > > quote char we should surely only allow it on input - just because other > > programs produce absurd output there is not reason we should. > > Yeah. The *real* problem with the patch as proposed is that it allows a > COPY OUT to emit a file that cannot be reloaded correctly, even given > the same options used to prepare it. I think that the restrictions were > put there more to prevent that scenario than to restrict COPY IN. erp. My apologies, I hadn't ever intended for this to be used with COPY OUT. For some reason I had thought my changes were isolated to the COPY CSV IN path. I'd be happy to adjust the patch to only accept the quote-in-null syntax when doing a COPY CSV IN. Thanks, Stephen
Attachment:
signature.asc
Description: Digital signature