Hello,
This is about:
http://archives.postgresql.org/pgsql-patches/2007-11/msg00028.php
In order to resolve <function name>.<parameter name> the check in
transformColumnRef goes into case 2 which is A.B
The ideal way to resolve the func.param in this case is to perform the
callback (hook)
after ALL the checks (like qualifiedNameToVar) are performed.
Unfortunately for func.param is this problematic because
qualifiedNameToVar and transformWholeRowRef/ParseFuncOrColumn
internally call RTE functions which justifiably throw an "ambiguous column
error".
I would like your opinion about the following implementations:
Only when parsing functions and only in case of A.B:
a) Check for func.param before qualifiedNameToVar and transformWholeRowRef/ParseFuncOrColumn
result: func.param resolves correctly, but: in case of a function name/parameter name being the same as a
tablename / column name, the func.param will be resolved first. for example:
create table func( id int, fld1 varchar );
create function func( id int, fld1 varchar ) ... as $$ select func.id from func where func.id
=func.id; $$ language sql;
above will produce the unexpected result hence <function
name>.<parameter name> will be resolved first.
b) step 1. call qualifiedNameToVar with implicitRTEOK = FALSE; step 2. if no results then perform callback; step 3.
Nextif no results then qualifiedNameToVar with
implicitRTEOK = TRUE (default); step 4. if no result then continue as ParseFuncOrColumn
result: func.param resolves correctly but: the order will be qualifiedNameToVar , CallBack , ParseFuncOrColumn.
I have tested both options (a) and (b)
Current regression test + new tests pass on Win and RH.
Any thoughts?
Regards,
Gevik.
------------------------------------------------
Gevik Babakhani
PostgreSQL NL http://www.postgresql.nl
TrueSoftware BV http://www.truesoftware.nl
------------------------------------------------
