From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Pavel Stehule <pavel(dot)stehule(at)hotmail(dot)com> |
Cc: | tgl(at)sss(dot)pgh(dot)pa(dot)us, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] proposal - plpgsql: execute using into |
Date: | 2006-08-21 17:45:54 |
Message-ID: | 200608211745.k7LHjsg03125@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Pavel Stehule wrote:
> > > There are some problems about replacing string values in the SQL string.
> >
> >Doesn't the Oracle implementation already imply a solution to that?
> >
>
> I don't know. I didn't find any detail documentation about it. I don't know
> what Oracle exactly do.
Oracle does use USING:
EXECUTE IMMEDIATE dynamic_string
[INTO {define_variable[, define_variable]... | record}]
[USING [IN | OUT | IN OUT] bind_argument
[, [IN | OUT | IN OUT] bind_argument]...]
[{RETURNING | RETURN} INTO bind_argument[, bind_argument]...];
so I think we are OK there.
> >I think we'd be best off to leave EXECUTE alone, at least until we've
> >converged to the point where almost nobody is using non-standard-compliant
> >strings.
> >
>
> Maybe, but patch have to solve SQL string and non SQL strings too
The only case I see you using it is for \:. What is the purpose of
that? Can't we use :: for a literal :?
I have attached the patch from March.
--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Attachment | Content-Type | Size |
---|---|---|
execute_using.dif | text/x-diff | 22.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2006-08-21 17:52:26 | Re: ISBN/ISSN/ISMN/EAN13 module |
Previous Message | Markus Schiltknecht | 2006-08-21 17:42:47 | Re: Replication |
From | Date | Subject | |
---|---|---|---|
Next Message | Marko Kreen | 2006-08-21 18:10:10 | Re: [PATCHES] [PATCH] Provide 8-byte transaction IDs to |
Previous Message | Tom Lane | 2006-08-21 17:29:48 | Re: [PATCHES] [PATCH] Provide 8-byte transaction IDs to |