Re: V0.2 patch for TODO Item: SQL-language referenceparameters by name.

From: "Gevik Babakhani" <pgdev(at)xs4all(dot)nl>
To: "'Gregory Stark'" <stark(at)enterprisedb(dot)com>, "'David Fetter'" <david(at)fetter(dot)org>
Cc: "'Tom Lane'" <tgl(at)sss(dot)pgh(dot)pa(dot)us>, <pgsql-patches(at)postgresql(dot)org>
Subject: Re: V0.2 patch for TODO Item: SQL-language referenceparameters by name.
Date: 2007-11-03 18:04:08
Message-ID: 000e01c81e43$f1be93e0$0a01a8c0@gevmus
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

So where do we go from here?

a. <function name>.<arg name>
b. <this>.<arg name>
c. ':'<argname>
d. just <argname>

option a,b and d are easy to implement.
option d would be least clear and readable considering
sql functions can be long and have multiple arguments.

option c is more difficult because gram.y has to be modified
to understand ':'<identifier> as parameter but not a target_list item.

option a and b would make the source more readable
but extra documentation has to be provided to describe
how to refer arguments by name.

Regards,
Gevik

------------------------------------------------
Gevik Babakhani

PostgreSQL NL http://www.postgresql.nl
TrueSoftware BV http://www.truesoftware.nl
------------------------------------------------

> -----Original Message-----
> From: pgsql-patches-owner(at)postgresql(dot)org
> [mailto:pgsql-patches-owner(at)postgresql(dot)org] On Behalf Of Gregory Stark
> Sent: Saturday, November 03, 2007 6:22 PM
> To: David Fetter
> Cc: Tom Lane; Gevik Babakhani; pgsql-patches(at)postgresql(dot)org
> Subject: Re: [PATCHES] V0.2 patch for TODO Item: SQL-language
> referenceparameters by name.
>
>
> "David Fetter" <david(at)fetter(dot)org> writes:
>
> > What I mean by "kinda" is that it's a standard way of handling
> > parameters in Oracle and in DBI.
>
> That's a good reason *not* to use them for other purposes.
> Users trying to create procedures through DBI or other
> interfaces like it will run into problems when the driver
> misinterprets the parameters.
>
> > I think it would be a very bad idea
> > to require that people use the function name in parameters,
>
> I think were talking about only allowing it to disambiguate
> if the name is shadowed by a variable in an inner scope.
>
> --
> Gregory Stark
> EnterpriseDB http://www.enterprisedb.com
> Ask me about EnterpriseDB's RemoteDBA services!
>
> ---------------------------(end of
> broadcast)---------------------------
> TIP 1: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo(at)postgresql(dot)org
> so that your
> message can get through to the mailing list cleanly
>

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2007-11-03 18:07:49 Re: V0.2 patch for TODO Item: SQL-language reference parameters by name.
Previous Message Gregory Stark 2007-11-03 17:21:52 Re: V0.2 patch for TODO Item: SQL-language referenceparameters by name.