postgres.git / summary / log / commit / refs

commit    4bd371f6f886446b6c0886f088992e6f3d4ab4a2
Author:   Kevin Grittner <kgrittn@postgresql.org>
Date:     Sat Nov 30 17:24:56 2013 +0000

    Fix pg_dumpall to work for databases flagged as read-only.
    
    pg_dumpall's charter is to be able to recreate a database cluster's
    contents in a virgin installation, but it was failing to honor that
    contract if the cluster had any ALTER DATABASE SET
    default_transaction_read_only settings.  By including a SET command
    for the connection for each connection opened by pg_dumpall output,
    errors are avoided and the source cluster is successfully
    recreated.
    
    There was discussion of whether to also set this for the connection
    applying pg_dump output, but it was felt that it was both less
    appropriate in that context, and far easier to work around.
    
    Backpatch to all supported branches.


 src/bin/pg_dump/pg_dumpall.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c
index 336ae588c7d..05c96462f38 100644
--- a/src/bin/pg_dump/pg_dumpall.c
+++ b/src/bin/pg_dump/pg_dumpall.c
@@ -452,6 +452,9 @@ main(int argc, char *argv[])
 	 * database we're connected to at the moment is fine.
 	 */
 
+	/* Restore will need to write to the target cluster */
+	fprintf(OPF, "SET default_transaction_read_only = off;\n\n");
+
 	/* Replicate encoding and std_strings in output */
 	fprintf(OPF, "SET client_encoding = '%s';\n",
 			pg_encoding_to_char(encoding));
@@ -1598,6 +1601,17 @@ dumpDatabases(PGconn *conn)
 
 		fprintf(OPF, "\\connect %s\n\n", fmtId(dbname));
 
+		/*
+		 * Restore will need to write to the target cluster.  This connection
+		 * setting is emitted for pg_dumpall rather than in the code also used
+		 * by pg_dump, so that a cluster with databases or users which have
+		 * this flag turned on can still be replicated through pg_dumpall
+		 * without editing the file or stream.  With pg_dump there are many
+		 * other ways to allow the file to be used, and leaving it out allows
+		 * users to protect databases from being accidental restore targets.
+		 */
+		fprintf(OPF, "SET default_transaction_read_only = off;\n\n");
+
 		if (filename)
 			fclose(OPF);
 


[parent: 34fa72ec9cfc]