Re: POC PATCH: copy from ... exceptions to: (was Re: VLDB Features)

FWIW, Greenplum has a similar construct (but which also logs the errors
in the
db) where data type errors are skipped as long as the number of errors
don't
exceed a reject limit.  If the reject limit is reached then the COPY
fails:
>
>       LOG ERRORS [ SEGMENT REJECT LIMIT <count> [ ROWS | PERCENT ]]
>
IIRC the gist of this was to catch then the user copies the wrong input
data or
plain has a broken file.  Rather than finding out after copying n rows
which
are likely to be garbage the process can be restarted.

This version of the patch has a compiler error in the error message:

I felt just logging "Error: %ld" would make people wonder the meaning of
the %ld. Logging something like ""Error: %ld data type errors were
found" might be clearer.