Re: EXECUTE USING for plpgsql (for 8.4) - Mailing list pgsql-patches

From Heikki Linnakangas
Subject Re: EXECUTE USING for plpgsql (for 8.4)
Date
Msg-id 471E5707.4090203@enterprisedb.com
Whole thread Raw
In response to Re: EXECUTE USING for plpgsql (for 8.4)  ("Pavel Stehule" <pavel.stehule@gmail.com>)
Responses Re: EXECUTE USING for plpgsql (for 8.4)  ("Pavel Stehule" <pavel.stehule@gmail.com>)
List pgsql-patches
Pavel Stehule wrote:
>> This doesn't work:
>>
>> create function exc_using(varchar) returns varchar
>> as $$
>>  declare v varchar;
>> begin
>>   execute 'select upper($1)' into v using ('aa');
>
> it cannot work. Your parameter is row.

Really? "execute 'select upper($1)' into v using ('aa'::varchar);"
works, as does "execute 'select $1 + 1' into v using (12345);".

> But into v using 'aaa' doesn't work too :(
>
> ERROR:  failed to find conversion function from unknown to text
> CONTEXT:  SQL statement "select upper($1)"
>
> you have to specify type: use argument, variable or casting
> .... using text 'aaa'; or select upper($1::text)
>
> It is question for Tom. Why prepared statement cannot cast from literal to text
> http://www.nabble.com/Blowback-from-text-conversion-changes-t3977711.html

Yeah, I suppose we'll just live with that. Using literals as arguments
is kind of pointless anyway, since you could as well put the literal in
the query as well and not bother with the USING.

>> I also noted that the patch makes USING a keyword. Not sure if we care
>> about that or not.
>>
> I am afraid to change well know syntax (SQL/PSM use it in same context too).

No I think the syntax is fine. I'm just wondering if it really has to be
a reserved keyword to implement that syntax. Looking at the plpgsql
grammar close, we don't categorize keywords like we do in the main
grammar, so maybe what I'm saying doesn't make any sense.

--
  Heikki Linnakangas
  EnterpriseDB   http://www.enterprisedb.com

pgsql-patches by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: Win32: Minimising desktop heap usage
Next
From: Tom Lane
Date:
Subject: Re: uuid.h on Debian