From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Brendan Jurd <direvus(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: WIP: to_char, support for EEEE format |
Date: | 2009-04-10 17:51:23 |
Message-ID: | 162867790904101051r664d8effr432e350310ec0455@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
I know very well, so all texts in my patches should be translated to
English. My language skills are really minimal.
So, please, if you can, propose these error messages (with hints)-
result will be much better.
Thank you
Pavel
2009/4/10 Brendan Jurd <direvus(at)gmail(dot)com>:
> On Sat, Apr 11, 2009 at 2:16 AM, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
>>
>> I was surprised so PostgreSQL doesn't support this basic output format.
>>
>
> Hi Pavel,
>
> I had a look through your patch and would like to suggest improvements
> to the new error messages you've introduced.
>
> 1. "invalid using of format EEEE"
>
> This message occurs several times in the patch. For one thing, the
> grammar is wrong; it should be "use", not "using".
>
> Additionally, this message on its own is not very helpful. If I was
> trying to use to_char and got "invalid use of format" my first thought
> would be "Invalid how?" The message should at minimum have a DETAIL,
> and possibly a HINT as well to make it effective.
>
> 2. "cannot use EEEE and others"
>
> The wording on this message is a bit awkward. I think what you meant
> to say is that EEEE cannot be used with certain other formatting
> codes, but this should be made explicit in the message.
>
> Cheers,
> BJ
>
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2009-04-10 18:21:23 | Re: Re: [BUGS] BUG #4027: backslash escaping notdisabled inplpgsql |
Previous Message | Brendan Jurd | 2009-04-10 17:42:22 | Re: WIP: to_char, support for EEEE format |