Re: 8.1 substring bug? - Mailing list pgsql-hackers

From Tom Lane
Subject Re: 8.1 substring bug?
Date
Msg-id 19228.1131725252@sss.pgh.pa.us
Whole thread Raw
In response to Re: 8.1 substring bug?  (Stephan Szabo <sszabo@megazone.bigpanda.com>)
Responses Re: 8.1 substring bug?
List pgsql-hackers
Stephan Szabo <sszabo@megazone.bigpanda.com> writes:
> It looks to me like we should be supporting any exact numeric with scale 0
> there (at least AFAICS from SQL92 and SQL03), so I don't think the current
> behavior is compliant. It doesn't look like adding a numeric overload
> of the function works, and the function also becomes ambiguous for int2
> inputs. :(

Currently (see gram.y, about line 7600) the grammar converts
SUBSTRING(foo FOR bar)

into
pg_catalog.substring(foo, 1, bar)

and then leaves the normal function-call-analysis code to make the best
of it with that.  If "bar" isn't implicitly castable to integer then
you've got trouble.

I was toying with the idea of making it translate instead to
pg_catalog.substring(foo, 1, (bar)::int4)

since AFAICS there isn't any other reasonable mapping once you have
committed to having the "1" in there.  This does not solve the general
problem, but it'd address the particular case anyway ...
        regards, tom lane


pgsql-hackers by date:

Previous
From: Martijn van Oosterhout
Date:
Subject: Re: 8.1 substring bug?
Next
From: Tom Lane
Date:
Subject: Re: 8.1 substring bug?