Re: [PATCH] Allow multiple recursive self-references

Поиск
Список
Период
Сортировка
От Zhihong Yu
Тема Re: [PATCH] Allow multiple recursive self-references
Дата
Msg-id CALNJ-vSXdj-gvkpTdSQw8EMM-Nax7ibJ8HzC_SDbYhk5HENHyg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] Allow multiple recursive self-references  (Denis Hirn <denis.hirn@uni-tuebingen.de>)
Ответы Re: [PATCH] Allow multiple recursive self-references
Список pgsql-hackers
Дерево обсуждения


On Tue, Aug 17, 2021 at 5:58 AM Denis Hirn <denis.hirn@uni-tuebingen.de> wrote:
> The tests fail when you build with assertions enabled (configure --enable-cassert).

Thank you for pointing that out. The new version of this patch fixes that.
The tests are working properly now. All style related issues are fixed as well.

Best wishes,
  -- Denis


Hi,
+                   selfrefcountL = cstate->selfrefcount;
+                   cstate->selfrefcount = selfrefcount;

Maybe the variable  selfrefcountL can be renamed slightly (e.g. curr_selfrefcount) so that the code is easier to read.

Cheers

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

Предыдущее
От: Zhihong Yu
Дата:
Сообщение: Re: Allow parallel DISTINCT
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: PG14: Avoid checking output-buffer-length for every encoded byte during pg_hex_encode