Re: pgsql: Prevent instability in contrib/pageinspect's regression test.

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: pgsql: Prevent instability in contrib/pageinspect's regression test.
Дата
Msg-id 3219281.1669054138@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: pgsql: Prevent instability in contrib/pageinspect's regression test.  (Robert Haas <robertmhaas@gmail.com>)
Ответы Re: pgsql: Prevent instability in contrib/pageinspect's regression test.
Список pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes:
> On Mon, Nov 21, 2022 at 12:35 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> Hmm, so the problem is:
>> 
>> SELECT octet_length(get_raw_page('test1', 'main', 0)) AS main_0;
>> ERROR:  cannot access temporary tables during a parallel operation
>> 
>> Why in the world is get_raw_page() marked as parallel safe?
>> It clearly isn't, given this restriction.

> I suspect that restriction was overlooked when evaluating the marking
> of this function.

So it would seem.  PARALLEL RESTRICTED should work, though.

I'll check to see if any sibling functions have the same issue,
and push a patch to adjust them.

Presumably the parallel labeling has to be fixed as far back as
it's marked that way (didn't look).  Maybe we should push the
test change further back too, just to exercise this.

            regards, tom lane



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

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: pgsql: Prevent instability in contrib/pageinspect's regression test.
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: Catalog_xmin is not advanced when a logical slot is lost