On 13.06.22 04:32, Julien Rouhaud wrote:
I think it makes more sense to have a sanity check to prevent HEADER
MATCH with COPY TO.
I'm fine with it. I added such a check and mentioned it in the documentation.
I think it would still be problematic if the target table has dropped columns.
Fortunately, as I initially thought the problem is only due to a thinko in the
original commit which used a wrong variable for the raw_fields offset. Once
fixed (attached v1) I didn't see any other problem in the rest of the logic and
all the added regression tests work as expected.
Thanks for this patch. I'll check it in detail in a bit. It looks good
to me at first glance.
