On Wed, Dec 18, 2024 at 7:45 PM David Rowley <dgrowle...@gmail.com> wrote: > On Wed, 18 Dec 2024 at 22:29, Richard Guo <guofengli...@gmail.com> wrote: > > Should we be concerned about passing a NULL TupleTableSlotOps in > > nodeRecursiveUnion.c? > > I made it pass NULL on purpose because the slot type on the recursive > union can be different on the inner and outer sides. Do you see issues > with that?
I see. I didn't notice any real issues with that; I was just flagged by the XXX comment there, which raises the question of whether it's worth working harder to determine the inputOps. Thanks Richard
