From: Alvaro Herrera <alvherre@alvh.no-ip.org>
Date: Thu, 24 Jan 2019 14:18:35 +0000 (-0300)
Subject: Simplify coding to detach constraints when detaching partition
X-Git-Tag: REL_12_BETA1~879
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=19184fcc09739abf75ccdada965ed6135c6d07c3;p=thirdparty%2Fpostgresql.git

Simplify coding to detach constraints when detaching partition

The original coding was too baroque and led to an use-after-release
mistake, noticed by buildfarm member prion.

Discussion: https://postgr.es/m/21693.1548305934@sss.pgh.pa.us
---

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 738c1781078..887c19c3eff 100644
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -15106,24 +15106,14 @@ ATExecDetachPartition(Relation rel, RangeVar *name)
 		idx = index_open(idxid, AccessExclusiveLock);
 		IndexSetParentIndex(idx, InvalidOid);
 		update_relispartition(classRel, idxid, false);
-		index_close(idx, NoLock);
-
-		/*
-		 * Detach any constraints associated with the index too.  Only UNIQUE
-		 * and PRIMARY KEY index constraints can be inherited, so no need
-		 * to check for others.
-		 */
-		if (!idx->rd_index->indisprimary && !idx->rd_index->indisunique)
-			continue;
 
+		/* If there's a constraint associated with the index, detach it too */
 		constrOid = get_relation_idx_constraint_oid(RelationGetRelid(partRel),
 													idxid);
-		if (!OidIsValid(constrOid))
-			elog(ERROR, "missing pg_constraint entry of index \"%s\" of partition \"%s\"",
-				 RelationGetRelationName(idx),
-				 RelationGetRelationName(partRel));
+		if (OidIsValid(constrOid))
+			ConstraintSetParentConstraint(constrOid, InvalidOid);
 
-		ConstraintSetParentConstraint(constrOid, InvalidOid);
+		index_close(idx, NoLock);
 	}
 	table_close(classRel, RowExclusiveLock);