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" > Subject: [PATCH v9 2/3] Add backlinks to progress reporting documentation I think these should say that they report their progress *in* the view (not "to"): > + Each backend running <command>ANALYZE</command> will report its progress > to > + the <structname>pg_stat_progress_analyze</structname> view. See > + Each backend running <command>CLUSTER</command> will report its progress > to > + the <structname>pg_stat_progress_cluster</structname> view. See > + Each backend running <command>COPY</command> will report its progress to > + the <structname>pg_stat_progress_copy</structname> view. See > + Each backend running <command>CREATE INDEX</command> will report its > + progress to the <structname>pg_stat_progress_create_index</structname> > + Each backend running <command>REINDEX</command> will report its progress > + to the <structname>pg_stat_progress_create_index</structname> view. See Like this one: > + Each backend running <command>VACUUM</command> without the > + <literal>FULL</literal> option will report its progress in the I'm sorry I didn't include that in last week's message. You could also write: |"The progress of each backend running >ANALYZE< is reported in the >pg_stat_progress_analyze< view." 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 -- Justin
