From 6dc3fd90f126fa409eac51a4182ee09b9d63c16c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=81lvaro=20Herrera?= Date: Mon, 28 Sep 2026 17:41:38 +0200 Subject: [PATCH] Turn "check" into assertion In commit 1813f951cad5, I mistakenly assumed that one particular function named CheckSomething() checks Something. It doesn't, it merely return the state to check. Fix by adding the necessary Assert() call. Good chance to add trailers for commit 1813f951cad5: Reviewed-by: Manuel Reyes Bravo Reviewed-by: Antonin Houska Reviewed-by: Masahiko Sawada Backpatch-through: 19 Discussion: https://postgr.es/m/CAA-aLv5MF6BLL+BWvix2Yw+CBardtH43AofPReQunhDZPNBtuA@mail.gmail.com --- src/backend/commands/repack.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backend/commands/repack.c b/src/backend/commands/repack.c index d7585312e17..596c1abaf78 100644 --- a/src/backend/commands/repack.c +++ b/src/backend/commands/repack.c @@ -1455,8 +1455,8 @@ copy_table_data(Relation NewHeap, Relation OldHeap, Relation OldIndex, if (!concurrent) LockRelationOid(OldHeap->rd_rel->reltoastrelid, lmode); else - CheckRelationOidLockedByMe(OldHeap->rd_rel->reltoastrelid, - lmode, false); + Assert(CheckRelationOidLockedByMe(OldHeap->rd_rel->reltoastrelid, + lmode, false)); } /*