On Wed, Feb 24, 2021 at 1:23 PM Justin Pryzby <pry...@telsasoft.com> wrote: > > On Sun, Feb 21, 2021 at 08:10:09PM +0100, Matthias van de Meent wrote: > > Subject: [PATCH v9 1/3] Add progress-reported components for COPY progress > > reporting > > > /* Increment amount of processed tuples and update > > the progress */ > > /* Increment amount of processed tuples and update the progress */ > > Ideally, this would say "number of processed tuples"
Correct. It's introduced by the original COPY progress reporting patch. Having said that, we could just correct them in the 0001 patch. > Looking at the existing docs: > > https://www.postgresql.org/docs/devel/progress-reporting.html#COPY-PROGRESS-REPORTING > | OID of the table on which the COPY command is executed > > Maybe it should say ".. is executing". Or ".. being executed": > | OID of the table on which the COPY command is being executed +1 for changing it to "OID of the table on which the COPY command is being executed." With Regards, Bharath Rupireddy. EnterpriseDB: http://www.enterprisedb.com
