| From: | Sami Imseih <samimseih(at)gmail(dot)com> |
|---|---|
| To: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
| Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, SATYANARAYANA NARLAPURAM <satyanarlapuram(at)gmail(dot)com> |
| Subject: | Re: Report index currently being vacuumed in pg_stat_progress_vacuum |
| Date: | 2026-08-19 18:31:02 |
| Message-ID: | CAA5RZ0vV_+NwqCjvkgz8MzxOy15MeqFTmR19GbYcCGEeQ2M81g@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> In short, I tend to agree with having one row per worker in the vacuum
> progress report, joining pg_stat_activity's leader_pid for simpler
> usability, extensibility, and less information bloat, along with doc
> changes to explain this. One concern is that some progress fields
> would be null on worker rows, but documenting this should be
> sufficient. I could be missing something here, so I would like to hear
> some thoughts before coming up with a patch.
>
I will look at the rest of the points later in detail, but it does sound
like to me that a new worker view will be a better place to hold extra per
worker ( or leader ) details and the current view will remain high
level/progress data. Unused progress fields do not sound right to me, and I
also worry we will bloat the existing view over time if we want to add more
per worker fields.
--
Sami
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bharath Rupireddy | 2026-08-19 18:44:00 | Re: Further cleanup related to statistics import support in postgres_fdw |
| Previous Message | Andrey Rachitskiy | 2026-08-19 18:22:34 | Re: Residual cleanups for tied objects in PL/Perl |