Bruce Momjian <pgman@candle.pha.pa.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.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?

               http://archives.postgresql.org

Reply via email to