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
signature.asc
Description: PGP signature
