From: Justin Pryzby <pry...@telsasoft.com> > I think this change to the regression tests is suspicous: > > -CONTEXT: remote SQL command: INSERT INTO public.loc2(f1, f2) VALUES > ($1, $2) > -COPY rem2, line 1: "-1 xyzzy" > +CONTEXT: COPY loc2, line 1: "-1 xyzzy" > +remote SQL command: COPY public.loc2(f1, f2) FROM STDIN > +COPY rem2, line 2 > > I think it shouldn't say "COPY rem2, line 2" but rather a remote version of > the > same: > |COPY loc2, line 1: "-1 xyzzy"
No, the output is OK. The remote message is included as the first line of the CONTEXT message field. The last line of the CONTEXT field is something that was added by the local COPY command. (Anyway, useful enough information is included in the message -- the constraint violation and the data that caused it.) > I have rebased this on my side over yesterday's libpq changes - I'll send it > if > you want, but it's probably just as easy if you do it. I've managed to rebased it, although it took unexpectedly long. The patch is attached. It passes make check against core and postgres_fdw. I'll turn the CF status back to ready for committer shortly. Regards Takayuki Tsunakawa
v19-0001-Fast-COPY-FROM-into-the-foreign-or-sharded-table.patch
Description: v19-0001-Fast-COPY-FROM-into-the-foreign-or-sharded-table.patch
