From: | Joe Conway <mail(at)joeconway(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-patches(at)postgresql(dot)org |
Subject: | Re: contrib/dblink regression test failure fix |
Date: | 2002-09-23 17:42:58 |
Message-ID: | 3D8F52A2.1010706@joeconway.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Tom Lane wrote:
> Actually, I was going to suggest modifying the declaration of the
> dblink function(s) to take integer instead of int2. Is there a
> really good reason why they take int2? If not, there's little point
> in putting up with a notational headache throughout the 7.3 cycle.
>
> The problem should go away again in 7.4, when we tweak the parser to
> initially type "2" as "2::int2" ... but for now the path of least
> resistance would seem to be avoiding declaring functions to take int2.
Good point. I only used int2 to be consistent with pg_class.relnatts.
Here is a revised patch. This uses an int4 input with a bit of checking to be
sure the given value fits in an int2 variable. This seemed the easiest and
safest approach. Please apply if there are no objections.
Thanks,
Joe
Attachment | Content-Type | Size |
---|---|---|
dblink-regr-cast-fix.2.patch | text/plain | 6.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2002-09-23 18:11:53 | Re: contrib/dblink regression test failure fix |
Previous Message | Alvaro Herrera | 2002-09-23 17:33:50 | Re: Implementation of LIMIT on DELETE and UPDATE statements |