Skip site navigation (1) Skip section navigation (2)

Re: [PATCHES] allow CSV quote in NULL

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>,Gregory Stark <stark(at)enterprisedb(dot)com>,PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] allow CSV quote in NULL
Date: 2007-07-31 18:52:39
Message-ID: 20070731185239.GP4887@tamriel.snowman.net (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-patches
* 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

In response to

pgsql-hackers by date

Next:From: Peter EisentrautDate: 2007-07-31 19:06:35
Subject: Re: stats_block_level
Previous:From: Greg SmithDate: 2007-07-31 18:51:03
Subject: Re: Machine available for community use

pgsql-patches by date

Next:From: Gregory StarkDate: 2007-07-31 19:09:56
Subject: Re: Export user visible function to make use of convert_to_scalar
Previous:From: Tom LaneDate: 2007-07-31 17:31:06
Subject: Re: Export user visible function to make use of convert_to_scalar

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group