Re: Should pg_stat_bgwriter.buffers_backend_fsync be removed?

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Peter Geoghegan <pg(at)heroku(dot)com>
Cc: Bernd Helmle <mailings(at)oopsware(dot)de>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Should pg_stat_bgwriter.buffers_backend_fsync be removed?
Date: 2014-04-29 13:24:30
Message-ID: CA+TgmobJmHCSMO7ue-O58b25OPHxSv3nCKhYV52Fg8f5ATw=DQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Apr 29, 2014 at 5:54 AM, Peter Geoghegan <pg(at)heroku(dot)com> wrote:
> On Tue, Apr 29, 2014 at 2:51 AM, Bernd Helmle <mailings(at)oopsware(dot)de> wrote:
>>> I suggest removing it for 9.5, and instead logging individual
>>> occurrences of backend fsync requests within ForwardFsyncRequest(). It
>>> seems fair to treat that as an anomaly to draw particular attention
>>> to.
>>
>> But wouldn't that make it more complicated/unlikely to discover cases, where
>> it still doesn't work?
>
> I don't think so, no.

I think it just depends. For people who are running a log scraper
anyway, a message would be better than a statistics counter, because
it's one less thing to check. For people who are running something
that monitors the stats views anyway, but perhaps not a log scraper,
the counter is better.

Overall, I don't see much reason to tinker with this. If we had no
reporting at all of this condition now, I'd probably be mildly more
supportive of adding a log message than a counter. But since we've
already got something and there's no real problem with it, I'm
disinclined to make a change.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2014-04-29 13:30:08 Re: Should pg_stat_bgwriter.buffers_backend_fsync be removed?
Previous Message Greg Stark 2014-04-29 11:41:23 Re: Problem with displaying "wide" tables in psql