From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Greg Smith <greg(at)2ndquadrant(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Count backend self-sync calls |
Date: | 2010-11-15 17:48:58 |
Message-ID: | AANLkTik+2f=BYdkUZdySbZrfZ4r1Va1QJHH92C77qR+j@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Nov 14, 2010 at 8:31 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Sun, Nov 14, 2010 at 7:19 PM, Greg Smith <greg(at)2ndquadrant(dot)com> wrote:
>>> But if this is generating a lot of log data or adding a lot of
>>> overhead, then you have bigger problems anyway:
>>>
>>> + elog(DEBUG1, "Unable to forward fsync request, executing
>>> directly");
>>>
>>
>> The argument against this log line even existing is that if the field is
>> added to pg_stat_bgwriter, that's probably how you want to monitor this data
>> anyway.
>
> I'll remove it if you really want it gone, but personally I think it's
> useful to have. I've more than once had to debug a problem given a
> PostgreSQL log file with the debug level cranked up and not a whole
> lot else. Rare events that cause performance to tank are worth
> logging, IMHO.
>
>> I started out touching code that called it just "sync", but then crossed to
>> other code that called it "fsync", and made the external UI use that name.
>> No objections to sorting that out within my patch so it's consistent.
>
> OK, I'll do that before I commit it.
Committed with (I think) all the changes discussed, plus a catversion bump.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2010-11-15 18:16:45 | Re: Re: Rethinking hint bits WAS: Protecting against unexpected zero-pages: proposal |
Previous Message | Aidan Van Dyk | 2010-11-15 17:45:25 | Re: unlogged tables |