Justin Pryzby <pry...@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