Opinion / advice needed for TODO: function params ref by name
От | Gevik Babakhani |
---|---|
Тема | Opinion / advice needed for TODO: function params ref by name |
Дата | |
Msg-id | 001c01c82090$fabae170$0a01a8c0@gevmus обсуждение исходный текст |
Ответы |
Re: Opinion / advice needed for TODO: function params ref by name
|
Список | pgsql-hackers |
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 ------------------------------------------------
В списке pgsql-hackers по дате отправления: