Re: [BUGS] BUG #2221: Bad delimiters allowed in COPY ...

From: David Fetter <david(at)fetter(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net>, neilc(at)samurai(dot)com, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [BUGS] BUG #2221: Bad delimiters allowed in COPY ...
Date: 2006-02-01 06:56:05
Message-ID: 20060201065605.GC9577@fetter.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-patches

On Wed, Feb 01, 2006 at 01:16:08AM -0500, Tom Lane wrote:
> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > Attached is a patch that errors for \r and \n in delimiter and
> > null. I kept the ERRCODE_FEATURE_NOT_SUPPORTED error code because
> > that is what all the other error tests use in the copy code in
> > that area.
>
> I'd go with INVALID_PARAMETER_VALUE, I think. ISTM that
> FEATURE_NOT_SUPPORTED is appropriate for places where we might
> someday support the case the error is rejecting. For instance the
> error just above your patch is for a multi-character delimiter
> string. That isn't completely senseless, it's just not implemented.
> But we're not ever going to allow a delimiter setting that conflicts
> with end-of-line, and I don't foresee allowing some other value for
> end-of-line ;-) ... so this check isn't going to be removed someday.

I don't know why you're saying that the EOL character will never be
changeable. Other DBs (yes, I know that's not an argument for doing
this, but please bear with me) let you set the "field separator" aka
our DELIMITER and "record separator" aka our newline (or CRLF, in some
cases. Oy!).

Anyhow, Bruce's patch still allows backslash as a delimiter, which can
cause *all* kinds of fun if not disallowed.

Cheers,
D
--
David Fetter david(at)fetter(dot)org http://fetter.org/
phone: +1 415 235 3778

Remember to vote!

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Andrew Dunstan 2006-02-01 12:55:39 Re: [BUGS] BUG #2221: Bad delimiters allowed in COPY ...
Previous Message Tom Lane 2006-02-01 06:16:08 Re: [BUGS] BUG #2221: Bad delimiters allowed in COPY ...

Browse pgsql-patches by date

  From Date Subject
Next Message Simon Riggs 2006-02-01 07:12:07 Re: BTree vacuum before page splitting
Previous Message Tom Lane 2006-02-01 06:16:08 Re: [BUGS] BUG #2221: Bad delimiters allowed in COPY ...