Re: pltcl - "Cache lookup for attribute" error - version

Jan Wieck wrote:
> Tom Lane wrote:
> 
> > Jan Wieck  writes:
> >> When assigning a tuple to an array, PL/Tcl creates one extra array 
> >> element .tupno telling the SPI_tuptable index of the result tuple. I 
> >> think I originally planned to have more of these critters ... but 
> >> probably never really needed them. It is in there since 6.3!
> > 
> >> Bottom line is, if one has a trigger, and inside the trigger he does an 
> >> SPI_exec, fetches a tuple into an array and then returns [array get x] 
> >> instead of new or old ... so from the back through the right chest into 
> >> the left eye ... then it will fail if the .tupno isn't filtered out.
> > 
> > Hm.  Perhaps we should tighten the test to reject only ".tupno", rather
> > than any name starting with dot?
> 
> Man you have worries ... aren't people who use identifiers with a 
> leading dot supposed to have problems? What about changing it to .. 
> instead? I mean, how does such a thing look like?
> 
>      SELECT ".. some column .." FROM ".. the schema ..".".. a table .."
>          WHERE ".. the schema ..".".. a table ..".".. some column .."
>          IN ('.oh.', '.give.', '.me.', '.a.', '.break!');
> 
> If you like to, tighten it.

Jan, if I understand correctly, I agree with Tom.  It seems strange to
add a restriction on indentifiers in pl/tcl that doesn't exist in any of
the other interfaces.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073