Re: Add mode column to pg_stat_progress_vacuum
| От | Masahiko Sawada |
|---|---|
| Тема | Re: Add mode column to pg_stat_progress_vacuum |
| Дата | |
| Msg-id | CAD21AoC8k9kP9zdhwK91z9SFt5E1-J+O6Oq_zpDwnoQ4J4gJ=Q@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: Add mode column to pg_stat_progress_vacuum (Shinya Kato <shinya11.kato@gmail.com>) |
| Ответы |
Re: Add mode column to pg_stat_progress_vacuum
|
| Список | pgsql-hackers |
On Mon, Nov 24, 2025 at 6:06 PM Shinya Kato <shinya11.kato@gmail.com> wrote: > > On Tue, Nov 25, 2025 at 8:13 AM Masahiko Sawada <sawada.mshk@gmail.com> wrote: > > > What about “started_by” ? it’s unambiguous and consistent with other columns > > > like “query_start” in pg_stat_activity. > > > > "started_by" sounds reasonable to me. > > Thanks all. I've updated the patches to use "started_by" and replaced > remaining uses of "triggered" with "started". Thank you for updating the patches! I've attached a small change to simplify the 0001 patch. Please review it. For the 0002 patch adding "started_by" column to pg_stat_progress_analyze system view, while we can easily check that the analyze operation reported in pg_stat_progress_analyze is started manually or by autovacuum worker by joining it to pg_stat_activity on the pid column, the proposed column would help users to check it even easier. So +1. 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". --- + <command>ANALYZE</command> or <command>VACUUM (ANALYZE)</command> + command. How about using "... or VACUUM with the ANALYZE option"? Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
Вложения
В списке pgsql-hackers по дате отправления: