From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Gregory Stark <stark(at)enterprisedb(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: pgsql: Add quotes to message |
Date: | 2009-02-23 12:56:01 |
Message-ID: | 49A29CE1.4000208@gmx.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Gregory Stark wrote:
> petere(at)postgresql(dot)org (Peter Eisentraut) writes:
>
>> Log Message:
>> -----------
>> Add quotes to message
>
> errdetail("Returned type %s does not match expected type "
> ! "%s in column \"%s\".",
> OidIsValid(returned->attrs[i]->atttypid) ?
> format_type_be(returned->attrs[i]->atttypid) :
> _(dropped_column_type),
>
> I'm surprised there isn't a regression test for this case.
Me too actually. ;-) In general, the plpgsql test doesn't appear to
cover the error paths very well at all. I changed the wording of about
50 error messages the other day and only two or three of those appeared
in a regression test case.
(The test coverage framework doesn't cover the PLs yet, so I don't have
anything more precise to say on that.)
From | Date | Subject | |
---|---|---|---|
Next Message | User Bmomjian | 2009-02-23 14:02:13 | pg-migrator - src: Use assert rather than Assert because we don't link |
Previous Message | Gregory Stark | 2009-02-23 10:45:22 | Re: pgsql: Add quotes to message |
From | Date | Subject | |
---|---|---|---|
Next Message | Grzegorz Jaskiewicz | 2009-02-23 13:43:24 | Re: 8.4 features presentation |
Previous Message | Gregory Stark | 2009-02-23 10:45:22 | Re: pgsql: Add quotes to message |