| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | Michael Paquier <michael(at)paquier(dot)xyz> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Ronan Dunklau <ronan(dot)dunklau(at)aiven(dot)io>, Önder Kalacı <onderkalaci(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: A potential memory leak on Merge Join when Sort node is not below Materialize node |
| Date: | 2022-09-28 23:34:51 |
| Message-ID: | CAApHDvr1XhLBH7EZue1USduOV97SnXbd_ujA=LhVASSGkU+4VQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, 29 Sept 2022 at 12:30, Michael Paquier <michael(at)paquier(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2022-09-28 23:35:07 | Re: A potential memory leak on Merge Join when Sort node is not below Materialize node |
| Previous Message | Michael Paquier | 2022-09-28 23:30:31 | Re: A potential memory leak on Merge Join when Sort node is not below Materialize node |