From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
Cc: | Simon Riggs <simon(at)2ndquadrant(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: archive_command failures report confusing exit status |
Date: | 2007-12-12 00:59:10 |
Message-ID: | 9452.1197421150@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> Simon Riggs wrote:
>> I think you should lose the "The", if nothing else. Most Postgres
>> messages I recall say "return code = xx" not "The return code ..."
> Right, that's because they are in errmessage() and not errdetail(). The
> guidelines are different.
Right --- errdetail is supposed to be a complete sentence. (The
rationale for that is explained in TFM.) I do find the current phrasing
a bit awkward, though; what's bugging me is putting quotes around the
command string. It seems highly likely that the command string will
itself contain quotes and thus that the added quotes will be confusing.
I wonder if it'd be OK to do
The failed archive command was: %s
which is stretching the style guidelines by omitting a trailing period,
but I think that might be justifiable in this context.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Joshua D. Drake | 2007-12-12 02:18:32 | Slow PITR restore |
Previous Message | Tom Lane | 2007-12-12 00:43:22 | Re: archive_command failures report confusing exit status |