From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Tue, 3 Sep 2002 16:00:02 +0000 (+0000)
Subject: I think reindex_index had better get AccessExclusiveLock on the parent
X-Git-Tag: REL7_3~601
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a3c9ef766ba82cc44c826bf14b2e06e337560a33;p=postgresql

I think reindex_index had better get AccessExclusiveLock on the parent
heap, not just ExclusiveLock.
---

diff --git a/src/backend/catalog/index.c b/src/backend/catalog/index.c
index 489d1c607e..f91600f94a 100644
--- a/src/backend/catalog/index.c
+++ b/src/backend/catalog/index.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/catalog/index.c,v 1.194 2002/09/03 01:04:41 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/catalog/index.c,v 1.195 2002/09/03 16:00:02 tgl Exp $
  *
  *
  * INTERFACE ROUTINES
@@ -1818,7 +1818,7 @@ reindex_index(Oid indexId, bool force, bool inplace)
 	indexInfo = BuildIndexInfo(iRel->rd_index);
 
 	/* Open the parent heap relation */
-	heapRelation = heap_open(heapId, ExclusiveLock);
+	heapRelation = heap_open(heapId, AccessExclusiveLock);
 	if (heapRelation == NULL)
 		elog(ERROR, "reindex_index: can't open heap relation");
 
@@ -1919,8 +1919,8 @@ reindex_relation(Oid relid, bool force)
 
 	/*
 	 * Ensure to hold an exclusive lock throughout the transaction. The
-	 * lock could be less intensive but now it's AccessExclusiveLock for
-	 * simplicity.
+	 * lock could be less intensive (in the non-overwrite path) but now
+	 * it's AccessExclusiveLock for simplicity.
 	 */
 	rel = heap_open(relid, AccessExclusiveLock);