On Wed, Jul 2, 2025 at 4:48 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
> >> Regarding the documentation, how about explicitly stating that when MATCH is specified, only
> >> the first line is skipped? While this may seem obvious, it’s worth clarifying, as the semantics
> >> of the HEADER option have become a bit more complex with this change.
> >
> > Agreed. I have updated the documentation as follows:
> >
> > + lines are discarded. If the option is set to <literal>MATCH</literal>,
> > + the number and names of the columns in the header line must exactly
> > + match those of the table and, in order, after which the header line is
> > + discarded; otherwise an error is raised. The <literal>MATCH</literal>
>
> How about making the wording a bit clearer? For example:
>
> If set to MATCH, the first line is discarded, and it must contain column names that
> exactly match the table's columns, in both number and order; otherwise, an error is raised.
Thank you for the review. I fixed it.
>
> Also, the phrase "if this option is set to..." is repeated three times in the current text.
> For the second and third instances, we could simplify it to just "if set to...".
Agreed. However, for the sake of symmetry between "On output" and "On
input" and to maintain clarity between the paragraphs, I have omitted
"this option is" from the "On input" paragraph only.
--
Best regards,
Shinya Kato
NTT OSS Center
