Document issues with non-default tablespaces and pg_dumpall restores.

Backpatch documentation addition to 8.1.X.

diff --git a/doc/src/sgml/ref/pg_dumpall.sgml b/doc/src/sgml/ref/pg_dumpall.sgml
index 6cf1c0d538c8665b281f05cf1aaf7c7e48e3cf9d..e6de1ea64e5f80d3ad6031d793f3b0c9834db1a5 100644 (file)
--- a/doc/src/sgml/ref/pg_dumpall.sgml
+++ b/doc/src/sgml/ref/pg_dumpall.sgml
@@ -1,5 +1,5 @@
 
 
@@ -377,6 +377,13 @@ PostgreSQL documentation
    databases.
   
 
+  
+   pg_dumpall requires all needed
+   tablespace directories to exist before the restore or
+   database creation will fail for databases in non-default
+   locations.
+  
+
  
 
 

diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c
index ecbd15f28c447b036d09c7d175608289f9b56526..b49766507e2238d4f9d13d5f21c8df4c3367ac9c 100644 (file)
--- a/src/bin/pg_dump/pg_dumpall.c
+++ b/src/bin/pg_dump/pg_dumpall.c
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  *
- * $PostgreSQL: pgsql/src/bin/pg_dump/pg_dumpall.c,v 1.79 2006/06/07 22:24:45 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/pg_dump/pg_dumpall.c,v 1.80 2006/06/16 22:01:17 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -888,7 +888,15 @@ dumpCreateDB(PGconn *conn)
            appendPQExpBuffer(buf, " ENCODING = ");
            appendStringLiteralConn(buf, dbencoding, conn);
 
-           /* Output tablespace if it isn't default */
+           /*
+            *  Output tablespace if it isn't the default.  For default, it
+            *  uses the default from the template database.  If tablespace
+            *  is specified and tablespace creation failed earlier,
+            *  (e.g. no such directory), the database creation will fail
+            *  too.  One solution would be to use 'SET default_tablespace'
+            *  like we do in pg_dump for setting non-default database
+            *  locations.
+            */
            if (strcmp(dbtablespace, "pg_default") != 0)
                appendPQExpBuffer(buf, " TABLESPACE = %s",
                                  fmtId(dbtablespace));