Justin Pryzby <pryzby@telsasoft.com> writes:
> On Fri, Mar 10, 2023 at 03:36:10PM -0500, Tom Lane wrote:
>> I took a look through this. It seems like basically a good solution,
>> but the count_leaf_partitions() function is bothering me, for two
>> reasons:
> ... find_all_inheritors() will also have been called by
> ProcessUtilitySlow(). Maybe it's sufficient to mention that ?
Hm. Could we get rid of count_leaf_partitions by doing the work in
ProcessUtilitySlow? Or at least passing that OID list forward instead
of recomputing it?
regards, tom lane