From ab2343d4cb806c43e8a7269d38b3bdddea185213 Mon Sep 17 00:00:00 2001 From: Peter Geoghegan Date: Wed, 29 Apr 2020 18:17:49 -0700 Subject: [PATCH] Remove redundant _bt_killitems() buffer check. _bt_getbuf() cannot return an invalid buffer. Oversight in commit 2ed5b87f96d. --- src/backend/access/nbtree/nbtutils.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/backend/access/nbtree/nbtutils.c b/src/backend/access/nbtree/nbtutils.c index ce48a51640a..84b391ce038 100644 --- a/src/backend/access/nbtree/nbtutils.c +++ b/src/backend/access/nbtree/nbtutils.c @@ -1756,10 +1756,6 @@ _bt_killitems(IndexScanDesc scan) /* Attempt to re-read the buffer, getting pin and lock. */ buf = _bt_getbuf(scan->indexRelation, so->currPos.currPage, BT_READ); - /* It might not exist anymore; in which case we can't hint it. */ - if (!BufferIsValid(buf)) - return; - page = BufferGetPage(buf); if (BufferGetLSNAtomic(buf) == so->currPos.lsn) so->currPos.buf = buf; -- 2.39.5