Re: Proposal TODO Item: SQL-language reference parameters by name - Mailing list pgsql-hackers

From Gevik Babakhani
Subject Re: Proposal TODO Item: SQL-language reference parameters by name
Date
Msg-id 011f01c81c05$5ed264c0$0a01a8c0@gevmus
Whole thread Raw
In response to Proposal TODO Item: SQL-language reference parameters by name  ("Gevik Babakhani" <pgdev@xs4all.nl>)
List pgsql-hackers
Thank you Tom.

After running a create function statement (language sql), the final check 
for a column is done in
parse_expr.c:transformColumnRef in case 1. Would this be the correct place 
to implement
functionality for a final match?

Regards,
Gevik.


----- Original Message ----- 
From: "Tom Lane" <tgl@sss.pgh.pa.us>
To: "Gevik Babakhani" <pgdev@xs4all.nl>
Cc: <pgsql-hackers@postgresql.org>
Sent: Wednesday, October 31, 2007 4:36 AM
Subject: Re: [HACKERS] Proposal TODO Item: SQL-language reference parameters 
by name


> "Gevik Babakhani" <pgdev@xs4all.nl> writes:
>>> I think the most likely implementation would involve adding hooks
>>> in the parser at places where "unknown column" errors are about to
>>> be thrown, so that a function language could check for a match to
>>> one of its variable names only after the query-exposed names are
>>> checked.
>
>> Would this be the right path to follow?
>
>> Assuming we are allowed to include parameter names into ParseState, we 
>> can
>> match the "non existing" colnames against the list of parameter
>> names when transformColumnRef (or someware safe in that path).
>
> You are assuming that the function language isn't interested in taking
> some extra action when a reference to a parameter is recognized.  This
> is demonstrably false for plpgsql, for one --- it wants to build a list
> of just which variables it will have to pass into each SQL command.
> A hook function can take care of that, a passive data structure can't.
>
> regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 7: You can help support the PostgreSQL project by donating at
>
>                http://www.postgresql.org/about/donate
> 



pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: A small rant about coding style for backend functions
Next
From: Tom Lane
Date:
Subject: Re: psql show dbsize?