Re: Remove pg_stat_progress_vacuum from Table 28.2
Amit Langote <langote_amit_f8@lab.ntt.co.jp>
From: Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>
To: Fujii Masao <masao.fujii@gmail.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>
Date: 2017-04-13T03:57:05Z
Lists: pgsql-hackers
On 2017/04/13 12:11, Fujii Masao wrote: > On Wed, Apr 12, 2017 at 10:32 AM, Amit Langote > <Langote_Amit_f8@lab.ntt.co.jp> wrote: >> On 2017/04/12 0:22, Fujii Masao wrote: >>> On Fri, Apr 7, 2017 at 9:53 AM, Amit Langote >>> <Langote_Amit_f8@lab.ntt.co.jp> wrote: >>>> On 2017/04/07 0:56, Fujii Masao wrote: >>>>> On Tue, Apr 4, 2017 at 10:19 AM, Amit Langote >>>>> <Langote_Amit_f8@lab.ntt.co.jp> wrote: >>>>>> It seems pg_stat_progress_vacuum is not supposed to appear in the table >>>>>> titled "Collected Statistics Views". It was added by c16dc1aca. Attached >>>>>> patch fixes that. >>>>> >>>>> Instead, it should appear in the table of "Dynamic Statistics Views" >>>>> because it reports dynamic info, i.e., progress, about VACUUM activity? >>>> >>>> I thought the same at first, but then realized we have a entirely separate >>>> section 28.4. Progress Reporting. >>> >>> Yes, but I don't think that removing that from 28.2 is an improvement >>> for users. >> >> Perhaps you're right. Here's a patch that moves pg_stat_progress_vacuum >> to the Dynamic Statistics Views table. > > Thanks for the patch! Pushed. Thanks, Regards, Amit
Commits
-
Move pg_stat_progress_vacuum to the table of Dynamic Statistics Views in doc.
- 5c63dab83836 9.6.3 landed
- 7a3e8d7b503c 10.0 landed