postgres.git / summary / log / commit / refs

commit    44f5bb245ce497ef052d93b5c17e368ffaac83a8
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Thu May 09 21:34:40 2013 +0000
First released in: 9.3.0, 9.1.10, 9.2.5

    Use pg_dump's --quote-all-identifiers option in pg_upgrade.
    
    This helps guard against changes in the set of reserved keywords from
    one version to another.  In theory it should only be an issue if we
    de-reserve a keyword in a newer release, since that can create the type
    of problem shown in bug #8128.
    
    Back-patch to 9.1 where the --quote-all-identifiers option was added.


 contrib/pg_upgrade/dump.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/contrib/pg_upgrade/dump.c b/contrib/pg_upgrade/dump.c
index 148dfb70c18..38169e6f90a 100644
--- a/contrib/pg_upgrade/dump.c
+++ b/contrib/pg_upgrade/dump.c
@@ -24,7 +24,8 @@ generate_old_dump(void)
 	 * restores the frozenid's for databases and relations.
 	 */
 	exec_prog(UTILITY_LOG_FILE, NULL, true,
-			  "\"%s/pg_dumpall\" %s --schema-only --binary-upgrade %s -f %s",
+			  "\"%s/pg_dumpall\" %s --schema-only --quote-all-identifiers "
+			  "--binary-upgrade %s -f %s",
 			  new_cluster.bindir, cluster_conn_opts(&old_cluster),
 			  log_opts.verbose ? "--verbose" : "",
 			  ALL_DUMP_FILE);


[parent: 3a33d5689b5e]