Remove UpdateFreeSpaceMap(), use FreeSpaceMapVacuumRange() instead. FreeSpaceMapVacuumRange has the same effect, is more efficient if many pages are involved, and makes fewer assumptions about how it's used. Notably, Claudio Freire pointed out that UpdateFreeSpaceMap could fail if the specified freespace value isn't the maximum possible. This isn't a problem for the single existing user, but the function represents an attractive nuisance IMO, because it's named as though it were a general-purpose update function and its limitations are undocumented. In any case we don't need multiple ways to get the same result.
In passing, do some code review and cleanup in RelationAddExtraBlocks. In particular, I see no excuse for it to omit the PageIsNew safety check that's done in the mainline extension path in RelationGetBufferForTuple. Discussion: https://postgr.es/m/cagtbqpyr0ujcntt3m5gozbrho+-gccno1ncaq8yejmzksw5...@mail.gmail.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/a063baaced273e955e088ba5979dcc6ec5cd92e6 Modified Files -------------- src/backend/access/heap/hio.c | 44 +++++++++++------ src/backend/storage/freespace/freespace.c | 81 ------------------------------- src/include/storage/freespace.h | 4 -- 3 files changed, 30 insertions(+), 99 deletions(-)