pgsql: Make create_unique_path manage memory like mark_dummy_rel. - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: Make create_unique_path manage memory like mark_dummy_rel.
Date
Msg-id E1eKQFp-0006sI-FL@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Make create_unique_path manage memory like mark_dummy_rel.

Put the unique path in the same context as the owning RelOptInfo, rather
than the toplevel planner context.  This is how this function worked
originally, but commit f41803bb39bc2949db200116a609fd242d0ec221
changed it without explanation.  mark_dummy_rel adopted the older (or
newer?) technique in commit eca75a12a27d28b972fc269c1c8813cd8eb15441,
which also featured a much better explanation of why it is correct.
So, switch back to that technique here, with the same explanation
given there.

Although this fixes a possible memory leak when GEQO is in use, the
leak is minor and probably nobody cares, so no back-patch.

Ashutosh Bapat, reviewed by Tom Lane and by me

Discussion: http://postgr.es/m/CAFjFpRcXkHHrXyD9BCvkgGJV4TnHG2SWJ0PhJfrDu3NAcQvh7g@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/1761653bbb17447906c812c347b3fe284ce699cf

Modified Files
--------------
src/backend/optimizer/util/pathnode.c | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Thomas Munro
Date:
Subject: Re: pgsql: Add some regression tests that exercise hash join code.
Next
From: Andres Freund
Date:
Subject: Re: pgsql: Add some regression tests that exercise hash join code.