| 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: | Whole Thread | Raw Message | 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 |
| 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. |