1berry/postgres
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't assume that PageIsEmpty() returns true on an all-zeros page.

Browse Source 
It does currently, and I don't see us changing that any time soon, but we
don't make that assumption anywhere else.

Per Tom Lane's suggestion. Backpatch to 9.2, like the previous patch that
added this assumption.
This commit is contained in:
Heikki Linnakangas 2015-07-27 18:54:09 +03:00
parent 746e7f1c18
commit 0e98ad0915
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/backend/access/spgist/spgvacuum.c
View File

@ -652,7 +652,7 @@ spgvacuumpage(spgBulkDeleteState *bds, BlockNumber blkno)
*/ */
if (!SpGistBlockIsRoot(blkno)) if (!SpGistBlockIsRoot(blkno))
{ {
if (PageIsEmpty(page)) if (PageIsNew(page) || PageIsEmpty(page))
{ {
RecordFreeIndexPage(index, blkno); RecordFreeIndexPage(index, blkno);
bds->stats->pages_deleted++; bds->stats->pages_deleted++;