On Aug 20, 2018, at 5:14 AM, Robert Haas <robertmhaas@gmail.com> wrote:
>
> On Sat, Aug 4, 2018 at 9:10 AM Surafel Temesgen <surafel3000@gmail.com> wrote:
> In order to prevent extreme condition the patch also add a new GUC variable called copy_max_error_limit that control
theamount of error to swallow before start to error and new failed record file options for copy to write a failed
recordso the user can examine it.
>
> Why should this be a GUC rather than a COPY option?
>
> In fact, try doing all of this by adding more options to COPY rather than new syntax.
>
> COPY ... WITH (ignore_conflicts 1000, ignore_logfile '...')
>
> It kind of stinks to use a log file written by the server as the dup-reporting channel though. That would have to be
superuser-only.
Perhaps a better option would be to allow the user to specify a name for a cursor, and have COPY do the moral
equivalentof DECLARE name? Calling a function for each bad row would be another option.