In pg_upgrade, clarify use of install_db_support_functions().

bmomjian · bmomjian · commit ebaf648ad2e4 · 2011-01-08T09:11:48.000-05:00
diff --git a/contrib/pg_upgrade/function.c b/contrib/pg_upgrade/function.c
@@ -13,13 +13,13 @@
 
 
 /*
- * install_db_support_functions()
+ * install_support_functions_in_db()
  *
  * pg_upgrade requires some support functions that enable it to modify
  * backend behavior.
  */
 void
-install_db_support_functions(const char *db_name)
+install_support_functions_in_db(const char *db_name)
 {
 	PGconn *conn = connectToServer(&new_cluster, db_name);
 	
diff --git a/contrib/pg_upgrade/pg_upgrade.c b/contrib/pg_upgrade/pg_upgrade.c
@@ -225,8 +225,11 @@ prepare_new_databases(void)
 
 	prep_status("Creating databases in the new cluster");
 
-	/* install support functions in the database used by GLOBALS_DUMP_FILE */
-	install_db_support_functions(os_info.user);
+	/*
+	 *	Install support functions in the database accessed by
+	 *	GLOBALS_DUMP_FILE because it can preserve pg_authid.oid.
+	 */
+	install_support_functions_in_db(os_info.user);
 
 	/*
 	 * We have to create the databases first so we can install support
@@ -261,7 +264,9 @@ create_new_objects(void)
 	{
 		DbInfo	   *new_db = &new_cluster.dbarr.dbs[dbnum];
 
-		install_db_support_functions(new_db->db_name);
+		/* skip db we already installed */
+		if (strcmp(new_db->db_name, os_info.user) != 0)
+			install_support_functions_in_db(new_db->db_name);
 	}
 	check_ok();
 
diff --git a/contrib/pg_upgrade/pg_upgrade.h b/contrib/pg_upgrade/pg_upgrade.h
@@ -321,7 +321,7 @@ void		check_hard_link(void);
 
 /* function.c */
 
-void		install_db_support_functions(const char *db_name);
+void		install_support_functions_in_db(const char *db_name);
 void		uninstall_support_functions(void);
 void		get_loadable_libraries(void);
 void		check_loadable_libraries(void);

Original file line number	Diff line number	Diff line change
`@@ -13,13 +13,13 @@`
`13`	`13`
`14`	`14`
`15`	`15`	`/*`
`16`		`- * install_db_support_functions()`
	`16`	`+ * install_support_functions_in_db()`
`17`	`17`	`*`
`18`	`18`	`* pg_upgrade requires some support functions that enable it to modify`
`19`	`19`	`* backend behavior.`
`20`	`20`	`*/`
`21`	`21`	`void`
`22`		`-install_db_support_functions(const char *db_name)`
	`22`	`+install_support_functions_in_db(const char *db_name)`
`23`	`23`	`{`
`24`	`24`	`PGconn *conn = connectToServer(&new_cluster, db_name);`
`25`	`25`