On 2018/02/09 14:31, Ashutosh Bapat wrote:
>> I also noticed that a later patch adds partsupfunc to PartitionScheme,
>> which the pruning patch needs too. So, perhaps would be nice to take out
>> that portion of the patch. That is, the changes to PartitionScheme struct
>> definition and those to find_partition_scheme().
>
> I am not sure whether a patch with just that change and without any
> changes to use that member will be acceptable. So leaving this aside.
I asked, because with everything that I have now changed in the partition
pruning patch, one would need to pass these FmgrInfo pointers down to
partition bound searching functions from the optimizer. If the changes to
add partsupfunc to the optimizer were taken out from your main patch, the
pruning patch could just start using it. For now, I'm making those
changes part of the pruning patch.
Thanks,
Amit