On Sat, 29 Mar 2025 at 05:54, Gurjeet Singh <gurjeet@singh.im> wrote: > > On Fri Mar 28, 2025 at 7:39 PM PDT, Tender Wang wrote: > > > In ExecInitPartitionInfo(), it has comments like: > > > > * This duplicates much of the logic in ExecInitMerge(), so something > > * changes there, look here too. > > > I would've expecetd the comment to be in ExecInitMerge(), asking the > reader to look at other places the logic is duplicated. >
There already is such a comment a little further up in ExecInitMerge(), so I didn't bother repeating it.
I've committed the fix, except that I put the test case in merge.sql, since part_abc_view doesn't exist in back branches of partition_prune.sql, and it feels more like a MERGE bug anyway.
