Re: [HACKERS] Arrays broken on temp tables

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [HACKERS] Arrays broken on temp tables
Дата
Msg-id 2954.941924013@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [HACKERS] Arrays broken on temp tables  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [HACKERS] Arrays broken on temp tables  (Bruce Momjian <maillist@candle.pha.pa.us>)
Список pgsql-hackers
> I'm betting that something in the array code is somehow bypassing the
> normal table lookup mechanism, and is managing to see the underlying
> temp-table name that should be hidden from it.  Will look further...

Yup, here it is, in parse_target.c:
   /*    * If there are subscripts on the target column, prepare an    * array assignment expression.  This will
generatean array value    * that the source value has been inserted into, which can then    * be placed in the new
tupleconstructed by INSERT or UPDATE.    * Note that transformArraySubscripts takes care of type coercion.    */   if
(indirection)  {       Attr       *att = makeNode(Attr);       Node       *arrayBase;       ArrayRef   *aref;
 
       att->relname = pstrdup(RelationGetRelationName(rd)->data);
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Next question is what to do about it --- the original table name
doesn't seem to be conveniently available in this routine.  A quick
search for other uses of RelationGetRelationName shows other places
that may have related bugs.  Possibly, temprel.c needs to provide
a reverse-lookup routine that will give back the user name of a table
that might be a temp table?
        regards, tom lane


В списке pgsql-hackers по дате отправления:

Предыдущее
От: Charles Tassell
Дата:
Сообщение: Re: [HACKERS] Re: [GENERAL] indexed regex select optimisation missing?
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: [HACKERS] Arrays broken on temp tables