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

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: David Fetter <david(at)fetter(dot)org>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Neil Conway <neilc(at)samurai(dot)com>, pgsql-bugs(at)postgresql(dot)org, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [PATCHES] BUG #2221: Bad delimiters allowed in COPY ...
Date: 2006-01-31 22:24:32
Message-ID: 200601312224.k0VMOWc18222@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-patches

> Is ERRCODE_FEATURE_NOT_SUPPORTED the right errcode? This isn't a
> missing feature; we are performing a sanity check here. We can
> reasonably expect never to support CR, LF or \ as the text
> delimiter.

I guess that depends on whether we ever plan to allow people to set
the output record separator to something other than CR?LF.

> Maybe ERRCODE_INVALID_PARAMETER_VALUE ? Or maybe we need a new one.

Agreed. Right now it is invalid and there are no plans to support other
values for end-of-line. I will make the change when the patch is
applied.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Bruce Momjian 2006-01-31 22:40:15 Re: [BUGS] BUG #2223: Misleading info in docs on volatility level of
Previous Message Bruce Momjian 2006-01-31 21:54:24 Re: BUG #2220: PostgreSQL-JDBC 8.1-404 fails to compile with

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2006-01-31 22:40:15 Re: [BUGS] BUG #2223: Misleading info in docs on volatility level of
Previous Message Bruce Momjian 2006-01-31 21:42:22 Re: pg_restore COPY error handling