Don't add a shared dependency on the owner of a composite type in pg_class.

alvherre · alvherre · commit 732a1fd1d08c · 2006-04-24T01:40:48.000Z
We track the owner in pg_type instead, as that is the place where the owner is
changed on ALTER TYPE ... OWNER TO.
diff --git a/src/backend/catalog/heap.c b/src/backend/catalog/heap.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/catalog/heap.c,v 1.295 2006/03/05 15:58:22 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/catalog/heap.c,v 1.296 2006/04/24 01:40:48 alvherre Exp $
  *
  *
  * INTERFACE ROUTINES
@@ -772,7 +772,13 @@ heap_create_with_catalog(const char *relname,
 		referenced.objectSubId = 0;
 		recordDependencyOn(&myself, &referenced, DEPENDENCY_NORMAL);
 
-		recordDependencyOnOwner(RelationRelationId, relid, ownerid);
+		/*
+		 * For composite types, the dependency on owner is tracked for the
+		 * pg_type entry, so don't record it here.  All other relkinds need
+		 * their ownership tracked.
+		 */
+		if (relkind != RELKIND_COMPOSITE_TYPE)
+			recordDependencyOnOwner(RelationRelationId, relid, ownerid);
 	}
 
 	/*

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/catalog/heap.c,v 1.295 2006/03/05 15:58:22 momjian Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/catalog/heap.c,v 1.296 2006/04/24 01:40:48 alvherre Exp $`
`12`	`12`	`*`
`13`	`13`	`*`
`14`	`14`	`* INTERFACE ROUTINES`
`@@ -772,7 +772,13 @@ heap_create_with_catalog(const char *relname,`
`772`	`772`	`referenced.objectSubId = 0;`
`773`	`773`	`recordDependencyOn(&myself, &referenced, DEPENDENCY_NORMAL);`
`774`	`774`
`775`		`- recordDependencyOnOwner(RelationRelationId, relid, ownerid);`
	`775`	`+ /*`
	`776`	`+ * For composite types, the dependency on owner is tracked for the`
	`777`	`+ * pg_type entry, so don't record it here. All other relkinds need`
	`778`	`+ * their ownership tracked.`
	`779`	`+ */`
	`780`	`+ if (relkind != RELKIND_COMPOSITE_TYPE)`
	`781`	`+ recordDependencyOnOwner(RelationRelationId, relid, ownerid);`
`776`	`782`	`}`
`777`	`783`
`778`	`784`	`/*`