From 0901b7c54fc3e7e8987fb61004dc557d16472ae5 Mon Sep 17 00:00:00 2001 From: Steven Niu Date: Tue, 13 Aug 2024 19:44:56 -0700 Subject: [PATCH] remove duplicated smgrclose Remove redundant `smgrclose` calls after `smgrdounlinkall`. The `smgrdounlinkall` now handles all necessary cleanup, making additional closes unnecessary. Author: Steven Niu Reviewed-by: Junwang Zhao Discussion: https://www.postgresql.org/message-id/flat/CABBtG=cDTCBDCBK7McSy6bJR3s5xUTOg0vSFfuW8oLdUYyCscA@mail.gmail.com --- src/backend/catalog/storage.c | 3 --- src/backend/storage/smgr/md.c | 2 -- src/backend/utils/cache/relcache.c | 1 - 3 files changed, 6 deletions(-) diff --git a/src/backend/catalog/storage.c b/src/backend/catalog/storage.c index 227df90f89c9..9f48cc9d5a88 100644 --- a/src/backend/catalog/storage.c +++ b/src/backend/catalog/storage.c @@ -727,9 +727,6 @@ smgrDoPendingDeletes(bool isCommit) { smgrdounlinkall(srels, nrels, false); - for (int i = 0; i < nrels; i++) - smgrclose(srels[i]); - pfree(srels); } } diff --git a/src/backend/storage/smgr/md.c b/src/backend/storage/smgr/md.c index f99c9d900134..117288a3dcea 100644 --- a/src/backend/storage/smgr/md.c +++ b/src/backend/storage/smgr/md.c @@ -1606,8 +1606,6 @@ DropRelationFiles(RelFileLocator *delrels, int ndelrels, bool isRedo) smgrdounlinkall(srels, ndelrels, isRedo); - for (i = 0; i < ndelrels; i++) - smgrclose(srels[i]); pfree(srels); } diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c index 18a14ae186ef..db2a013d1d5a 100644 --- a/src/backend/utils/cache/relcache.c +++ b/src/backend/utils/cache/relcache.c @@ -3802,7 +3802,6 @@ RelationSetNewRelfilenumber(Relation relation, char persistence) */ srel = smgropen(relation->rd_locator, relation->rd_backend); smgrdounlinkall(&srel, 1, false); - smgrclose(srel); } else {