postgres.git / summary / log / commit / refs

commit    588f50f851cdcb3da0755c3ad17a3427f1e57914
Author:   Heikki Linnakangas <heikki.linnakangas@iki.fi>
Date:     Mon Jul 27 15:54:09 2015 +0000

    Don't assume that PageIsEmpty() returns true on an all-zeros page.
    
    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.


 src/backend/access/spgist/spgvacuum.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/access/spgist/spgvacuum.c b/src/backend/access/spgist/spgvacuum.c
index 36ba24de583..cffcbb5a53d 100644
--- a/src/backend/access/spgist/spgvacuum.c
+++ b/src/backend/access/spgist/spgvacuum.c
@@ -653,7 +653,7 @@ spgvacuumpage(spgBulkDeleteState *bds, BlockNumber blkno)
 	 */
 	if (!SpGistBlockIsRoot(blkno))
 	{
-		if (PageIsEmpty(page))
+		if (PageIsNew(page) || PageIsEmpty(page))
 		{
 			RecordFreeIndexPage(index, blkno);
 			bds->stats->pages_deleted++;


[parent: bafe3b007301]