Re: Documentation about pg_stat_bgwriter

From: Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
To: peter(dot)eisentraut(at)2ndquadrant(dot)com
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Documentation about pg_stat_bgwriter
Date: 2017-05-19 03:23:20
Message-ID: 20170519.122320.208368854.horiguchi.kyotaro@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hello, thank you for the reply.

At Thu, 18 May 2017 20:48:44 -0400, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote in <343d4cdb-e25d-867d-2830-6502eca4df5c(at)2ndquadrant(dot)com>
> On 5/10/17 04:38, Kyotaro HORIGUCHI wrote:
> > Hi. While I read the documentation I found the following
> > description about some columns in pg_stat_bgwriter.
> >
> > https://www.postgresql.org/docs/devel/static/monitoring-stats.html#pg-stat-bgwriter-view
> >
> > This table shows cluster-global values, not per-backend values.
> >
> >> maxwritten_clean:
> >> Number of times the background writer stopped a cleaning scan
> >> because it had written too many buffers
> >> buffers_backend:
> >> Number of buffers written directly by a backend
> >> buffers_backend_fsync:
> >> Number of times a backend had to execute its own fsync call
> >> (normally the background writer handles those even when the
> >> backend does its own write)
> > Since the values are the summary in a cluster, the 'a backend's
> > in the last two seems wrong *to me*. I suppose the 'a backend'
> > should be just 'backends' or 'backends other than the background
> > writer' (This seems a bit verbose.).
>
> The text looks correct to me as it currently stands. Your alternative
> phrasings are also correct. But there is no need to change this, I think.

It's enough for me to know that it's not a kind of typo. Thanks.

regards,

--
Kyotaro Horiguchi
NTT Open Source Software Center

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2017-05-19 03:26:57 Re: [Proposal] Allow users to specify multiple tables in VACUUM commands
Previous Message Bossart, Nathan 2017-05-19 03:17:54 Re: [Proposal] Allow users to specify multiple tables in VACUUM commands