From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Greg Smith <greg(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Count backend self-sync calls |
Date: | 2010-11-14 23:46:16 |
Message-ID: | AANLkTik2ErwRZynB=v7O=nLuCmJP1tPZvSLm4qjGuFW8@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Nov 14, 2010 at 6:10 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>> With those changes, I think this is committable, and will do so,
>> barring objections.
>
> Obey message style guidelines, please, especially if you're going
> to promote any of those to ereports.
The only new message would be the one Greg has as:
Unable to forward fsync request, executing directly
For that, we could just go with:
could not forward fsync request
(Lower case, avoid use of unable, telegram style with program as
implicit subject.)
It might be even better to mention that the reason why we couldn't
forward the fsync request is that the fsync request queue is full.
I'm not sure exactly how to phrase that. I thought about:
fsync request queue is full
But that seems not to answer the "so what" question. There is an
example like this in the docs:
could not forward fsync request (fsync request queue is full)
...but I'm not sure I like that.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Joachim Wieland | 2010-11-14 23:48:07 | directory archive format for pg_dump |
Previous Message | David Fetter | 2010-11-14 23:31:12 | Re: SSI update |