David Rowley <david.row...@2ndquadrant.com> writes:
> I'm really hoping this is what you meant about the special-case code for
> Params.
> Does this look any better?
I'm starting to look this over and it seems like generally the right
thing, though I'm finding minor things I don't like much.
One thing I'm wondering about is why in the world are PartitionPruneInfo
and its subsidiary struct types declared in primnodes.h? They are not
general-purpose expression nodes, or if they are then there are an awful
lot of places that should know about them and do not. AFAICT they might
belong in plannodes.h.
regards, tom lane
