Re: Converting SetOp to read its two inputs separately
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: David Rowley <dgrowleyml@gmail.com>
Cc: Richard Guo <guofenglinux@gmail.com>, pgsql-hackers@lists.postgresql.org
Date: 2024-12-19T02:44:02Z
Lists: pgsql-hackers
Attachments
- v4-0001-Convert-SetOp-to-read-its-inputs-as-outerPlan-and.patch (text/x-diff) patch v4-0001
- v4-0002-Remove-some-dead-code-in-prepunion.c.patch (text/x-diff) patch v4-0002
- v4-0003-Get-rid-of-choose_hashed_setop.patch (text/x-diff) patch v4-0003
- v4-0004-Fix-bogus-decisions-about-whether-we-want-pre-sor.patch (text/x-diff) patch v4-0004
- v4-0005-Teach-generate_nonunion_paths-to-consider-pre-sor.patch (text/x-diff) patch v4-0005
With David's recent fixes to allow telling BuildTupleHashTableExt what input slot type to expect, it's possible to remove the per-row slot type conversions I was doing before. So here's an updated patchset with that done. The common_result_slot_type() function I wrote here perhaps should be made generally available, but I didn't do that yet. 0002-0005 are the same as before. regards, tom lane
Commits
-
Get rid of old version of BuildTupleHashTable().
- e0a2721f7c16 18.0 landed
-
Use ExecGetCommonSlotOps infrastructure in more places.
- f0b900086a37 18.0 landed
-
Improve planner's handling of SetOp plans.
- 8d96f57d5cc7 18.0 landed
-
Convert SetOp to read its inputs as outerPlan and innerPlan.
- 27627929528e 18.0 landed
-
Fix typo in header comment for set_operation_ordered_results_useful
- b6612aedc53a 18.0 landed
-
Allow planner to use Merge Append to efficiently implement UNION
- 66c0185a3d14 17.0 cited