Re: PHJ file leak. - Mailing list pgsql-hackers

From Tom Lane
Subject Re: PHJ file leak.
Date
Msg-id 26079.1573511085@sss.pgh.pa.us
Whole thread Raw
In response to Re: PHJ file leak.  (Thomas Munro <thomas.munro@gmail.com>)
Responses Re: PHJ file leak.  (Kyotaro Horiguchi <horikyota.ntt@gmail.com>)
Re: PHJ file leak.  (Kyotaro Horiguchi <horikyota.ntt@gmail.com>)
List 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 by date:

Previous
From: Thomas Munro
Date:
Subject: Re: PHJ file leak.
Next
From: Mark Dilger
Date:
Subject: Re: Missing dependency tracking for TableFunc nodes