On Tue, Aug 24, 2021 at 12:01:33PM +0200, Peter Eisentraut wrote:
> While analyzing this again, I think I found an existing mistake. The
> handling of RELKIND_PARTITIONED_INDEX in RelationGetNumberOfBlocksInFork()
> seems to be misplaced. See attached patch.
Right. This maps with RELKIND_HAS_STORAGE(). Makes me wonder whether
is would be better to add a check on RELKIND_HAS_STORAGE() in this
area, even if that's basically the same as the Assert() already used
in this code path.
--
Michael