Thank you for the review!

On Thu, Dec 4, 2025 at 9:15 AM Masahiko Sawada <[email protected]> wrote:
> I've attached a small change to simplify the 0001 patch. Please review it.

LGTM, and I've updated in v9 patches.

> Here are a few comments:
>
> +        <listitem>
> +         <para>
> +          <literal>manual</literal>: The analyze was started by an explicit
>
> For consistency with "started_by" in pg_stat_progress_vacuum, I think
> it's better to start with "The operation was started by".

I think "started_by" in pg_stat_progress_vacuum uses "The vacuum was
started by ...".

> ---
> +          <command>ANALYZE</command> or <command>VACUUM (ANALYZE)</command>
> +          command.
>
> How about using "... or VACUUM with the ANALYZE option"?

Agreed, I've fixed it.


-- 
Best regards,
Shinya Kato
NTT OSS Center

Attachment: v9-0001-Add-mode-and-started_by-columns-to-pg_stat_progre.patch
Description: Binary data

Attachment: v9-0002-Add-started_by-column-to-pg_stat_progress_analyze.patch
Description: Binary data

Reply via email to