Re: Add header support to text format and matching feature

From: Rémi Lapeyre <remi(dot)lapeyre(at)lenstra(dot)fr>
To: Zhihong Yu <zyu(at)yugabyte(dot)com>
Cc: David Steele <david(at)pgmasters(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>, vignesh C <vignesh21(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Daniel Verite <daniel(at)manitou-mail(dot)org>
Subject: Re: Add header support to text format and matching feature
Date: 2021-04-11 14:35:12
Message-ID: 8964EEB0-7AE1-4549-BA4C-119EF8CD266C@lenstra.fr
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

>
> >> This now reads "Represents whether the header must be absent, present or match.”.
>
> Since match shouldn't be preceded with be, I think we can say:
>
> Represents whether the header must match, be absent or be present.

Thanks, here’s a v10 version of the patch that fixes this.

Attachment Content-Type Size
v10-0001-Add-header-support-to-COPY-TO-text-format.patch application/octet-stream 7.4 KB
v10-0002-Add-header-matching-mode-to-COPY-FROM.patch application/octet-stream 14.6 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Joe Conway 2021-04-11 15:01:54 Re: PL/R regression on windows, but not linux with master.
Previous Message Tom Lane 2021-04-11 14:13:23 Re: PL/R regression on windows, but not linux with master.