Re: PHJ file leak.

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: PHJ file leak.
Дата
Msg-id 26079.1573511085@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: PHJ file leak.  (Thomas Munro <thomas.munro@gmail.com>)
Ответы Re: PHJ file leak.
Re: PHJ file leak.
Список pgsql-hackers
Thomas Munro <thomas.munro@gmail.com> writes:
> On Tue, Nov 12, 2019 at 1:24 AM Kyotaro Horiguchi
> <horikyota.ntt@gmail.com> wrote:
>> Hello. While looking a patch, I found that PHJ sometimes complains for
>> file leaks if accompanied by LIMIT.

> Thanks for the patch!  Yeah, this seems correct, but I'd like to think
> about it some more before committing.  I'm going to be a bit tied up
> with travel so that might be next week.

At this point we've missed the window for this week's releases, so
there's no great hurry (and it'd be best not to push any noncritical
patches into the back branches anyway, for the next 24 hours).

Although the patch seems unobjectionable as far as it goes, I'd like
to understand why we didn't see the need for it long since.  Is there
another call to ExecParallelHashCloseBatchAccessors somewhere, and
if so, is that one wrongly placed?

            regards, tom lane



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

Предыдущее
От: Thomas Munro
Дата:
Сообщение: Re: PHJ file leak.
Следующее
От: Mark Dilger
Дата:
Сообщение: Re: Missing dependency tracking for TableFunc nodes