Re: A potential memory leak on Merge Join when Sort node is not below Materialize node - Mailing list pgsql-hackers

From David Rowley
Subject Re: A potential memory leak on Merge Join when Sort node is not below Materialize node
Date
Msg-id CAApHDvr1XhLBH7EZue1USduOV97SnXbd_ujA=LhVASSGkU+4VQ@mail.gmail.com
Whole thread Raw
In response to Re: A potential memory leak on Merge Join when Sort node is not below Materialize node  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
On Thu, 29 Sept 2022 at 12:30, Michael Paquier <michael@paquier.xyz> wrote:
>
> On Thu, Sep 29, 2022 at 11:58:17AM +1300, David Rowley wrote:
> > I've just pushed the disable byref Datums patch I posted earlier. I
> > only made a small adjustment to make use of the TupleDescAttr() macro.
> > Önder, thank you for the report.
>
> Wouldn't it be better to have 3a58176 reflect the non-optimization
> path in the EXPLAIN output of a new regression test if none of the
> existing tests are able to show any difference?

There's nothing in EXPLAIN that shows that this optimization occurs.
Or, are you proposing that you think there should be something? and
for 15??

David



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: A potential memory leak on Merge Join when Sort node is not below Materialize node
Next
From: Tom Lane
Date:
Subject: Re: A potential memory leak on Merge Join when Sort node is not below Materialize node