]> granicus.if.org Git - postgresql/commitdiff
Provide for parallel restoration from a custom format archive. Each data and
authorAndrew Dunstan <andrew@dunslane.net>
Mon, 2 Feb 2009 20:07:37 +0000 (20:07 +0000)
committerAndrew Dunstan <andrew@dunslane.net>
Mon, 2 Feb 2009 20:07:37 +0000 (20:07 +0000)
post-data step is run in a separate worker child (a thread on Windows, a child
process elsewhere) up to the concurrent number specified by the new pg_restore
command-line --multi-thread | -m switch.

Andrew Dunstan, with some editing by Tom Lane.

doc/src/sgml/ref/pg_restore.sgml
src/bin/pg_dump/pg_backup.h
src/bin/pg_dump/pg_backup_archiver.c
src/bin/pg_dump/pg_backup_archiver.h
src/bin/pg_dump/pg_backup_custom.c
src/bin/pg_dump/pg_backup_db.c
src/bin/pg_dump/pg_backup_files.c
src/bin/pg_dump/pg_backup_null.c
src/bin/pg_dump/pg_backup_tar.c
src/bin/pg_dump/pg_dump.c
src/bin/pg_dump/pg_restore.c

index f13c644772242a819e815029e804724e78d5132d..92c9f4be935d9767f228f6dfdf4096ee6786c0f6 100644 (file)
@@ -1,4 +1,4 @@
-<!-- $PostgreSQL: pgsql/doc/src/sgml/ref/pg_restore.sgml,v 1.77 2009/01/05 16:54:36 tgl Exp $ -->
+<!-- $PostgreSQL: pgsql/doc/src/sgml/ref/pg_restore.sgml,v 1.78 2009/02/02 20:07:36 adunstan Exp $ -->
 
 <refentry id="APP-PGRESTORE">
  <refmeta>
       </listitem>
      </varlistentry>
 
+     <varlistentry>
+      <term><option>-m <replaceable class="parameter">number-of-threads</replaceable></option></term>
+      <term><option>--multi-thread=<replaceable class="parameter">number-of-threads</replaceable></option></term>
+      <listitem>
+       <para>
+        Run the most time-consuming parts of <application>pg_restore</>
+        &mdash; those which load data, create indexes, or create
+        constraints &mdash; using multiple concurrent connections to the
+        database. This option can dramatically reduce the time to restore a
+        large database to a server running on a multi-processor machine.
+       </para>
+
+       <para>
+        This option is ignored when emitting a script rather than connecting
+        directly to a database server.  Multiple threads cannot be used
+        together with <option>--single-transaction</option>.  Also, the input
+        must be a plain file (not, for example, a pipe), and at present only
+        the custom archive format is supported.
+       </para>
+      </listitem>
+     </varlistentry>
+
      <varlistentry>
       <term><option>-n <replaceable class="parameter">namespace</replaceable></option></term>
       <term><option>--schema=<replaceable class="parameter">schema</replaceable></option></term>
index fa7e2d55f13342008a423daedc0de079646ccfa2..72bc066853ee5788afe28b1eb7d343e5e496ad93 100644 (file)
@@ -15,7 +15,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup.h,v 1.48 2009/01/05 16:54:36 tgl Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup.h,v 1.49 2009/02/02 20:07:36 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -53,6 +53,14 @@ typedef enum _archiveMode
        archModeRead
 } ArchiveMode;
 
+typedef enum _teSection
+{
+       SECTION_NONE = 1,                       /* COMMENTs, ACLs, etc; can be anywhere */
+       SECTION_PRE_DATA,                       /* stuff to be processed before data */
+       SECTION_DATA,                           /* TABLE DATA, BLOBS, BLOB COMMENTS */
+       SECTION_POST_DATA                       /* stuff to be processed after data */
+} teSection;
+
 /*
  *     We may want to have some more user-readable data, but in the mean
  *     time this gives us some abstraction and type checking.
@@ -124,6 +132,7 @@ typedef struct _restoreOptions
        int                     suppressDumpWarnings;   /* Suppress output of WARNING entries
                                                                                 * to stderr */
        bool            single_txn;
+       int                     number_of_threads;
 
        bool       *idWanted;           /* array showing which dump IDs to emit */
 } RestoreOptions;
@@ -152,7 +161,8 @@ extern void ArchiveEntry(Archive *AHX,
                         const char *tag,
                         const char *namespace, const char *tablespace,
                         const char *owner, bool withOids,
-                        const char *desc, const char *defn,
+                        const char *desc, teSection section,
+                        const char *defn,
                         const char *dropStmt, const char *copyStmt,
                         const DumpId *deps, int nDeps,
                         DataDumperPtr dumpFn, void *dumpArg);
index 9d877cb2d32b42cbcee188825d63cd392ffd4616..076bb0bf6f12742bd30bde9fa385c6b01c81ae15 100644 (file)
@@ -15,7 +15,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.161 2009/01/13 11:44:56 mha Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_archiver.c,v 1.162 2009/02/02 20:07:36 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -24,8 +24,9 @@
 #include "dumputils.h"
 
 #include <ctype.h>
-
 #include <unistd.h>
+#include <sys/types.h>
+#include <sys/wait.h>
 
 #ifdef WIN32
 #include <io.h>
 
 #include "libpq/libpq-fs.h"
 
+/*
+ * Special exit values from worker children.  We reserve 0 for normal
+ * success; 1 and other small values should be interpreted as crashes.
+ */
+#define WORKER_CREATE_DONE             10
+#define WORKER_INHIBIT_DATA            11
+#define WORKER_IGNORED_ERRORS  12
+
+/*
+ * Unix uses exit to return result from worker child, so function is void.
+ * Windows thread result comes via function return.
+ */
+#ifndef WIN32
+#define parallel_restore_result void
+#else
+#define parallel_restore_result DWORD
+#endif
+
+/* IDs for worker children are either PIDs or thread handles */
+#ifndef WIN32
+#define thandle pid_t
+#else
+#define thandle HANDLE
+#endif
+
+typedef struct _restore_args
+{
+       ArchiveHandle *AH;
+       TocEntry      *te;
+} RestoreArgs;
+
+typedef struct _parallel_slot
+{
+       thandle         child_id;
+       RestoreArgs *args;
+} ParallelSlot;
+
+#define NO_SLOT (-1)
 
 const char *progname;
 
@@ -71,6 +110,30 @@ static void dumpTimestamp(ArchiveHandle *AH, const char *msg, time_t tim);
 static OutputContext SetOutput(ArchiveHandle *AH, char *filename, int compression);
 static void ResetOutput(ArchiveHandle *AH, OutputContext savedContext);
 
+static int restore_toc_entry(ArchiveHandle *AH, TocEntry *te,
+                                                        RestoreOptions *ropt, bool is_parallel);
+static void restore_toc_entries_parallel(ArchiveHandle *AH);
+static thandle spawn_restore(RestoreArgs *args);
+static thandle reap_child(ParallelSlot *slots, int n_slots, int *work_status);
+static bool work_in_progress(ParallelSlot *slots, int n_slots);
+static int get_next_slot(ParallelSlot *slots, int n_slots);
+static TocEntry *get_next_work_item(ArchiveHandle *AH,
+                                                                       TocEntry **first_unprocessed,
+                                                                       ParallelSlot *slots, int n_slots);
+static parallel_restore_result parallel_restore(RestoreArgs *args);
+static void mark_work_done(ArchiveHandle *AH, thandle worker, int status,
+                                                  ParallelSlot *slots, int n_slots);
+static void fix_dependencies(ArchiveHandle *AH);
+static void repoint_table_dependencies(ArchiveHandle *AH,
+                                                                          DumpId tableId, DumpId tableDataId);
+static void identify_locking_dependencies(TocEntry *te,
+                                                                                 TocEntry **tocsByDumpId);
+static void reduce_dependencies(ArchiveHandle *AH, TocEntry *te);
+static void mark_create_done(ArchiveHandle *AH, TocEntry *te);
+static void inhibit_data_for_failed_table(ArchiveHandle *AH, TocEntry *te);
+static ArchiveHandle *CloneArchive(ArchiveHandle *AH);
+static void DeCloneArchive(ArchiveHandle *AH);
+
 
 /*
  *     Wrapper functions.
@@ -131,7 +194,6 @@ RestoreArchive(Archive *AHX, RestoreOptions *ropt)
        TocEntry   *te;
        teReqs          reqs;
        OutputContext sav;
-       bool            defnDumped;
 
        AH->ropt = ropt;
        AH->stage = STAGE_INITIALIZING;
@@ -146,6 +208,7 @@ RestoreArchive(Archive *AHX, RestoreOptions *ropt)
         */
        if (ropt->create && ropt->dropSchema)
                die_horribly(AH, modulename, "-C and -c are incompatible options\n");
+
        /*
         * -1 is not compatible with -C, because we can't create a database
         *  inside a transaction block.
@@ -153,6 +216,21 @@ RestoreArchive(Archive *AHX, RestoreOptions *ropt)
        if (ropt->create && ropt->single_txn)
                die_horribly(AH, modulename, "-C and -1 are incompatible options\n");
 
+       /*
+        * Make sure we won't need (de)compression we haven't got
+        */
+#ifndef HAVE_LIBZ
+       if (AH->compression != 0 && AH->PrintTocDataPtr != NULL)
+       {
+               for (te = AH->toc->next; te != AH->toc; te = te->next)
+               {
+                       reqs = _tocEntryRequired(te, ropt, false);
+                       if (te->hadDumper && (reqs & REQ_DATA) != 0)
+                               die_horribly(AH, modulename, "cannot restore from compressed archive (compression not supported in this installation)\n");
+               }
+       }
+#endif
+
        /*
         * If we're using a DB connection, then connect it.
         */
@@ -268,148 +346,21 @@ RestoreArchive(Archive *AHX, RestoreOptions *ropt)
                 */
                if (AH->currSchema)
                        free(AH->currSchema);
-               AH->currSchema = strdup("");
+               AH->currSchema = NULL;
        }
 
        /*
-        * Now process each non-ACL TOC entry
+        * In serial mode, we now process each non-ACL TOC entry.
+        *
+        * In parallel mode, turn control over to the parallel-restore logic.
         */
-       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       if (ropt->number_of_threads > 1 && ropt->useDB)
+               restore_toc_entries_parallel(AH);
+       else
        {
-               AH->currentTE = te;
-
-               /* Work out what, if anything, we want from this entry */
-               reqs = _tocEntryRequired(te, ropt, false);
-
-               /* Dump any relevant dump warnings to stderr */
-               if (!ropt->suppressDumpWarnings && strcmp(te->desc, "WARNING") == 0)
-               {
-                       if (!ropt->dataOnly && te->defn != NULL && strlen(te->defn) != 0)
-                               write_msg(modulename, "warning from original dump file: %s\n", te->defn);
-                       else if (te->copyStmt != NULL && strlen(te->copyStmt) != 0)
-                               write_msg(modulename, "warning from original dump file: %s\n", te->copyStmt);
-               }
-
-               defnDumped = false;
-
-               if ((reqs & REQ_SCHEMA) != 0)   /* We want the schema */
-               {
-                       ahlog(AH, 1, "creating %s %s\n", te->desc, te->tag);
-
-                       _printTocEntry(AH, te, ropt, false, false);
-                       defnDumped = true;
-
-                       /*
-                        * If we could not create a table and --no-data-for-failed-tables
-                        * was given, ignore the corresponding TABLE DATA
-                        */
-                       if (ropt->noDataForFailedTables &&
-                               AH->lastErrorTE == te &&
-                               strcmp(te->desc, "TABLE") == 0)
-                       {
-                               TocEntry   *tes;
-
-                               ahlog(AH, 1, "table \"%s\" could not be created, will not restore its data\n",
-                                         te->tag);
-
-                               for (tes = te->next; tes != AH->toc; tes = tes->next)
-                               {
-                                       if (strcmp(tes->desc, "TABLE DATA") == 0 &&
-                                               strcmp(tes->tag, te->tag) == 0 &&
-                                               strcmp(tes->namespace ? tes->namespace : "",
-                                                          te->namespace ? te->namespace : "") == 0)
-                                       {
-                                               /* mark it unwanted */
-                                               ropt->idWanted[tes->dumpId - 1] = false;
-                                               break;
-                                       }
-                               }
-                       }
-
-                       /* If we created a DB, connect to it... */
-                       if (strcmp(te->desc, "DATABASE") == 0)
-                       {
-                               ahlog(AH, 1, "connecting to new database \"%s\"\n", te->tag);
-                               _reconnectToDB(AH, te->tag);
-                       }
-               }
-
-               /*
-                * If we have a data component, then process it
-                */
-               if ((reqs & REQ_DATA) != 0)
-               {
-                       /*
-                        * hadDumper will be set if there is genuine data component for
-                        * this node. Otherwise, we need to check the defn field for
-                        * statements that need to be executed in data-only restores.
-                        */
-                       if (te->hadDumper)
-                       {
-                               /*
-                                * If we can output the data, then restore it.
-                                */
-                               if (AH->PrintTocDataPtr !=NULL && (reqs & REQ_DATA) != 0)
-                               {
-#ifndef HAVE_LIBZ
-                                       if (AH->compression != 0)
-                                               die_horribly(AH, modulename, "cannot restore from compressed archive (compression not supported in this installation)\n");
-#endif
-
-                                       _printTocEntry(AH, te, ropt, true, false);
-
-                                       if (strcmp(te->desc, "BLOBS") == 0 ||
-                                               strcmp(te->desc, "BLOB COMMENTS") == 0)
-                                       {
-                                               ahlog(AH, 1, "restoring %s\n", te->desc);
-
-                                               _selectOutputSchema(AH, "pg_catalog");
-
-                                               (*AH->PrintTocDataPtr) (AH, te, ropt);
-                                       }
-                                       else
-                                       {
-                                               _disableTriggersIfNecessary(AH, te, ropt);
-
-                                               /* Select owner and schema as necessary */
-                                               _becomeOwner(AH, te);
-                                               _selectOutputSchema(AH, te->namespace);
-
-                                               ahlog(AH, 1, "restoring data for table \"%s\"\n",
-                                                         te->tag);
-
-                                               /*
-                                                * If we have a copy statement, use it. As of V1.3,
-                                                * these are separate to allow easy import from
-                                                * withing a database connection. Pre 1.3 archives can
-                                                * not use DB connections and are sent to output only.
-                                                *
-                                                * For V1.3+, the table data MUST have a copy
-                                                * statement so that we can go into appropriate mode
-                                                * with libpq.
-                                                */
-                                               if (te->copyStmt && strlen(te->copyStmt) > 0)
-                                               {
-                                                       ahprintf(AH, "%s", te->copyStmt);
-                                                       AH->writingCopyData = true;
-                                               }
-
-                                               (*AH->PrintTocDataPtr) (AH, te, ropt);
-
-                                               AH->writingCopyData = false;
-
-                                               _enableTriggersIfNecessary(AH, te, ropt);
-                                       }
-                               }
-                       }
-                       else if (!defnDumped)
-                       {
-                               /* If we haven't already dumped the defn part, do so now */
-                               ahlog(AH, 1, "executing %s %s\n", te->desc, te->tag);
-                               _printTocEntry(AH, te, ropt, false, false);
-                       }
-               }
-       }                                                       /* end loop over TOC entries */
+               for (te = AH->toc->next; te != AH->toc; te = te->next)
+                       (void) restore_toc_entry(AH, te, ropt, false);
+       }
 
        /*
         * Scan TOC again to output ownership commands and ACLs
@@ -457,6 +408,193 @@ RestoreArchive(Archive *AHX, RestoreOptions *ropt)
        }
 }
 
+/*
+ * Restore a single TOC item.  Used in both parallel and non-parallel restore;
+ * is_parallel is true if we are in a worker child process.
+ *
+ * Returns 0 normally, but WORKER_CREATE_DONE or WORKER_INHIBIT_DATA if
+ * the parallel parent has to make the corresponding status update.
+ */
+static int
+restore_toc_entry(ArchiveHandle *AH, TocEntry *te,
+                                 RestoreOptions *ropt, bool is_parallel)
+{
+       int         retval = 0;
+       teReqs          reqs;
+       bool            defnDumped;
+
+       AH->currentTE = te;
+
+       /* Work out what, if anything, we want from this entry */
+       reqs = _tocEntryRequired(te, ropt, false);
+
+       /* Dump any relevant dump warnings to stderr */
+       if (!ropt->suppressDumpWarnings && strcmp(te->desc, "WARNING") == 0)
+       {
+               if (!ropt->dataOnly && te->defn != NULL && strlen(te->defn) != 0)
+                       write_msg(modulename, "warning from original dump file: %s\n", te->defn);
+               else if (te->copyStmt != NULL && strlen(te->copyStmt) != 0)
+                       write_msg(modulename, "warning from original dump file: %s\n", te->copyStmt);
+       }
+
+       defnDumped = false;
+
+       if ((reqs & REQ_SCHEMA) != 0)   /* We want the schema */
+       {
+               ahlog(AH, 1, "creating %s %s\n", te->desc, te->tag);
+
+               _printTocEntry(AH, te, ropt, false, false);
+               defnDumped = true;
+
+               if (strcmp(te->desc, "TABLE") == 0)
+               {
+                       if (AH->lastErrorTE == te)
+                       {
+                               /*
+                                * We failed to create the table.
+                                * If --no-data-for-failed-tables was given,
+                                * mark the corresponding TABLE DATA to be ignored.
+                                *
+                                * In the parallel case this must be done in the parent,
+                                * so we just set the return value.
+                                */
+                               if (ropt->noDataForFailedTables)
+                               {
+                                       if (is_parallel)
+                                               retval = WORKER_INHIBIT_DATA;
+                                       else
+                                               inhibit_data_for_failed_table(AH, te);
+                               }
+                       }
+                       else
+                       {
+                               /*
+                                * We created the table successfully.  Mark the
+                                * corresponding TABLE DATA for possible truncation.
+                                *
+                                * In the parallel case this must be done in the parent,
+                                * so we just set the return value.
+                                */
+                               if (is_parallel)
+                                       retval = WORKER_CREATE_DONE;
+                               else
+                                       mark_create_done(AH, te);
+                       }
+               }
+
+               /* If we created a DB, connect to it... */
+               if (strcmp(te->desc, "DATABASE") == 0)
+               {
+                       ahlog(AH, 1, "connecting to new database \"%s\"\n", te->tag);
+                       _reconnectToDB(AH, te->tag);
+               }
+       }
+
+       /*
+        * If we have a data component, then process it
+        */
+       if ((reqs & REQ_DATA) != 0)
+       {
+               /*
+                * hadDumper will be set if there is genuine data component for
+                * this node. Otherwise, we need to check the defn field for
+                * statements that need to be executed in data-only restores.
+                */
+               if (te->hadDumper)
+               {
+                       /*
+                        * If we can output the data, then restore it.
+                        */
+                       if (AH->PrintTocDataPtr != NULL && (reqs & REQ_DATA) != 0)
+                       {
+                               _printTocEntry(AH, te, ropt, true, false);
+
+                               if (strcmp(te->desc, "BLOBS") == 0 ||
+                                       strcmp(te->desc, "BLOB COMMENTS") == 0)
+                               {
+                                       ahlog(AH, 1, "restoring %s\n", te->desc);
+
+                                       _selectOutputSchema(AH, "pg_catalog");
+
+                                       (*AH->PrintTocDataPtr) (AH, te, ropt);
+                               }
+                               else
+                               {
+                                       _disableTriggersIfNecessary(AH, te, ropt);
+
+                                       /* Select owner and schema as necessary */
+                                       _becomeOwner(AH, te);
+                                       _selectOutputSchema(AH, te->namespace);
+
+                                       ahlog(AH, 1, "restoring data for table \"%s\"\n",
+                                                 te->tag);
+
+                                       /*
+                                        * In parallel restore, if we created the table earlier
+                                        * in the run then we wrap the COPY in a transaction and
+                                        * precede it with a TRUNCATE.  If archiving is not on
+                                        * this prevents WAL-logging the COPY.  This obtains a
+                                        * speedup similar to that from using single_txn mode
+                                        * in non-parallel restores.
+                                        */
+                                       if (is_parallel && te->created)
+                                       {
+                                               /*
+                                                * Parallel restore is always talking directly to a
+                                                * server, so no need to see if we should issue BEGIN.
+                                                */
+                                               StartTransaction(AH);
+
+                                               /*
+                                                * If the server version is >= 8.4, make sure we issue
+                                                * TRUNCATE with ONLY so that child tables are not
+                                                * wiped.
+                                                */
+                                               ahprintf(AH, "TRUNCATE TABLE %s%s;\n\n",
+                                                                (PQserverVersion(AH->connection) >= 80400 ?
+                                                                 "ONLY " : ""),
+                                                                fmtId(te->tag));
+                                       }
+
+                                       /*
+                                        * If we have a copy statement, use it. As of V1.3,
+                                        * these are separate to allow easy import from
+                                        * withing a database connection. Pre 1.3 archives can
+                                        * not use DB connections and are sent to output only.
+                                        *
+                                        * For V1.3+, the table data MUST have a copy
+                                        * statement so that we can go into appropriate mode
+                                        * with libpq.
+                                        */
+                                       if (te->copyStmt && strlen(te->copyStmt) > 0)
+                                       {
+                                               ahprintf(AH, "%s", te->copyStmt);
+                                               AH->writingCopyData = true;
+                                       }
+
+                                       (*AH->PrintTocDataPtr) (AH, te, ropt);
+
+                                       AH->writingCopyData = false;
+
+                                       /* close out the transaction started above */
+                                       if (is_parallel && te->created)
+                                               CommitTransaction(AH);
+
+                                       _enableTriggersIfNecessary(AH, te, ropt);
+                               }
+                       }
+               }
+               else if (!defnDumped)
+               {
+                       /* If we haven't already dumped the defn part, do so now */
+                       ahlog(AH, 1, "executing %s %s\n", te->desc, te->tag);
+                       _printTocEntry(AH, te, ropt, false, false);
+               }
+       }
+
+       return retval;
+}
+
 /*
  * Allocate a new RestoreOptions block.
  * This is mainly so we can initialize it, but also for future expansion,
@@ -555,7 +693,8 @@ ArchiveEntry(Archive *AHX,
                         const char *namespace,
                         const char *tablespace,
                         const char *owner, bool withOids,
-                        const char *desc, const char *defn,
+                        const char *desc, teSection section,
+                        const char *defn,
                         const char *dropStmt, const char *copyStmt,
                         const DumpId *deps, int nDeps,
                         DataDumperPtr dumpFn, void *dumpArg)
@@ -578,6 +717,7 @@ ArchiveEntry(Archive *AHX,
 
        newToc->catalogId = catalogId;
        newToc->dumpId = dumpId;
+       newToc->section = section;
 
        newToc->tag = strdup(tag);
        newToc->namespace = namespace ? strdup(namespace) : NULL;
@@ -616,7 +756,7 @@ void
 PrintTOCSummary(Archive *AHX, RestoreOptions *ropt)
 {
        ArchiveHandle *AH = (ArchiveHandle *) AHX;
-       TocEntry   *te = AH->toc->next;
+       TocEntry   *te;
        OutputContext sav;
        char       *fmtName;
 
@@ -655,14 +795,22 @@ PrintTOCSummary(Archive *AHX, RestoreOptions *ropt)
 
        ahprintf(AH, ";\n;\n; Selected TOC Entries:\n;\n");
 
-       while (te != AH->toc)
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
        {
-               if (_tocEntryRequired(te, ropt, true) != 0)
+               if (ropt->verbose || _tocEntryRequired(te, ropt, true) != 0)
                        ahprintf(AH, "%d; %u %u %s %s %s %s\n", te->dumpId,
                                         te->catalogId.tableoid, te->catalogId.oid,
                                         te->desc, te->namespace ? te->namespace : "-",
                                         te->tag, te->owner);
-               te = te->next;
+               if (ropt->verbose && te->nDeps > 0)
+               {
+                       int             i;
+
+                       ahprintf(AH, ";\tdepends on:");
+                       for (i = 0; i < te->nDeps; i++)
+                               ahprintf(AH, " %d", te->dependencies[i]);
+                       ahprintf(AH, "\n");
+               }
        }
 
        if (ropt->filename)
@@ -1316,12 +1464,10 @@ getTocEntryByDumpId(ArchiveHandle *AH, DumpId id)
 {
        TocEntry   *te;
 
-       te = AH->toc->next;
-       while (te != AH->toc)
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
        {
                if (te->dumpId == id)
                        return te;
-               te = te->next;
        }
        return NULL;
 }
@@ -1696,9 +1842,9 @@ _allocAH(const char *FileSpec, const ArchiveFormat fmt,
        else
                AH->fSpec = NULL;
 
-       AH->currUser = strdup("");      /* So it's valid, but we can free() it later
-                                                                * if necessary */
-       AH->currSchema = strdup("");    /* ditto */
+       AH->currUser = NULL;            /* unknown */
+       AH->currSchema = NULL;          /* ditto */
+       AH->currTablespace = NULL;      /* ditto */
        AH->currWithOids = -1;          /* force SET */
 
        AH->toc = (TocEntry *) calloc(1, sizeof(TocEntry));
@@ -1734,10 +1880,6 @@ _allocAH(const char *FileSpec, const ArchiveFormat fmt,
        }
 #endif
 
-#if 0
-       write_msg(modulename, "archive format is %d\n", fmt);
-#endif
-
        if (fmt == archUnknown)
                AH->format = _discoverArchiveFormat(AH);
        else
@@ -1772,11 +1914,11 @@ _allocAH(const char *FileSpec, const ArchiveFormat fmt,
 void
 WriteDataChunks(ArchiveHandle *AH)
 {
-       TocEntry   *te = AH->toc->next;
+       TocEntry   *te;
        StartDataPtr startPtr;
        EndDataPtr      endPtr;
 
-       while (te != AH->toc)
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
        {
                if (te->dataDumper != NULL)
                {
@@ -1811,7 +1953,6 @@ WriteDataChunks(ArchiveHandle *AH)
                                (*endPtr) (AH, te);
                        AH->currToc = NULL;
                }
-               te = te->next;
        }
 }
 
@@ -1839,6 +1980,7 @@ WriteToc(ArchiveHandle *AH)
 
                WriteStr(AH, te->tag);
                WriteStr(AH, te->desc);
+               WriteInt(AH, te->section);
                WriteStr(AH, te->defn);
                WriteStr(AH, te->dropStmt);
                WriteStr(AH, te->copyStmt);
@@ -1868,8 +2010,7 @@ ReadToc(ArchiveHandle *AH)
        DumpId     *deps;
        int                     depIdx;
        int                     depSize;
-
-       TocEntry   *te = AH->toc->next;
+       TocEntry   *te;
 
        AH->tocCount = ReadInt(AH);
        AH->maxDumpId = 0;
@@ -1904,6 +2045,35 @@ ReadToc(ArchiveHandle *AH)
 
                te->tag = ReadStr(AH);
                te->desc = ReadStr(AH);
+
+               if (AH->version >= K_VERS_1_11)
+               {
+                       te->section = ReadInt(AH);
+               }
+               else
+               {
+                       /*
+                        * rules for pre-8.4 archives wherein pg_dump hasn't classified
+                        * the entries into sections
+                        */
+                       if (strcmp(te->desc, "COMMENT") == 0 ||
+                               strcmp(te->desc, "ACL") == 0)
+                               te->section = SECTION_NONE;
+                       else if (strcmp(te->desc, "TABLE DATA") == 0 ||
+                                        strcmp(te->desc, "BLOBS") == 0 ||
+                                        strcmp(te->desc, "BLOB COMMENTS") == 0)
+                               te->section = SECTION_DATA;
+                       else if (strcmp(te->desc, "CONSTRAINT") == 0 ||
+                                        strcmp(te->desc, "CHECK CONSTRAINT") == 0 ||
+                                        strcmp(te->desc, "FK CONSTRAINT") == 0 ||
+                                        strcmp(te->desc, "INDEX") == 0 ||
+                                        strcmp(te->desc, "RULE") == 0 ||
+                                        strcmp(te->desc, "TRIGGER") == 0)
+                               te->section = SECTION_POST_DATA;
+                       else
+                               te->section = SECTION_PRE_DATA;
+               }
+
                te->defn = ReadStr(AH);
                te->dropStmt = ReadStr(AH);
 
@@ -2269,13 +2439,15 @@ _reconnectToDB(ArchiveHandle *AH, const char *dbname)
         */
        if (AH->currUser)
                free(AH->currUser);
+       AH->currUser = NULL;
 
-       AH->currUser = strdup("");
-
-       /* don't assume we still know the output schema */
+       /* don't assume we still know the output schema, tablespace, etc either */
        if (AH->currSchema)
                free(AH->currSchema);
-       AH->currSchema = strdup("");
+       AH->currSchema = NULL;
+       if (AH->currTablespace)
+               free(AH->currTablespace);
+       AH->currTablespace = NULL;
        AH->currWithOids = -1;
 
        /* re-establish fixed state */
@@ -2304,7 +2476,6 @@ _becomeUser(ArchiveHandle *AH, const char *user)
         */
        if (AH->currUser)
                free(AH->currUser);
-
        AH->currUser = strdup(user);
 }
 
@@ -2824,15 +2995,13 @@ ReadHead(ArchiveHandle *AH)
  * checkSeek
  *       check to see if fseek can be performed.
  */
-
 bool
 checkSeek(FILE *fp)
 {
-
        if (fseeko(fp, 0, SEEK_CUR) != 0)
                return false;
        else if (sizeof(pgoff_t) > sizeof(long))
-
+       {
                /*
                 * At this point, pgoff_t is too large for long, so we return based on
                 * whether an pgoff_t version of fseek is available.
@@ -2842,6 +3011,7 @@ checkSeek(FILE *fp)
 #else
                return false;
 #endif
+       }
        else
                return true;
 }
@@ -2870,3 +3040,855 @@ dumpTimestamp(ArchiveHandle *AH, const char *msg, time_t tim)
                                 localtime(&tim)) != 0)
                ahprintf(AH, "-- %s %s\n\n", msg, buf);
 }
+
+
+/*
+ * Main engine for parallel restore.
+ *
+ * Work is done in three phases.
+ * First we process tocEntries until we come to one that is marked
+ * SECTION_DATA or SECTION_POST_DATA, in a single connection, just as for a
+ * standard restore.  Second we process the remaining non-ACL steps in
+ * parallel worker children (threads on Windows, processes on Unix), each of
+ * which connects separately to the database.  Finally we process all the ACL
+ * entries in a single connection (that happens back in RestoreArchive).
+ */
+static void
+restore_toc_entries_parallel(ArchiveHandle *AH)
+{
+       RestoreOptions *ropt = AH->ropt;
+       int                     n_slots = ropt->number_of_threads;
+       ParallelSlot *slots;
+       int                     work_status;
+       int                     next_slot;
+       TocEntry   *first_unprocessed = AH->toc->next;
+       TocEntry   *next_work_item;
+       thandle         ret_child;
+       TocEntry   *te;
+
+       ahlog(AH,2,"entering restore_toc_entries_parallel\n");
+
+       /* we haven't got round to making this work for all archive formats */
+       if (AH->ClonePtr == NULL || AH->ReopenPtr == NULL)
+               die_horribly(AH, modulename, "parallel restore is not supported with this archive file format\n");
+
+       slots = (ParallelSlot *) calloc(sizeof(ParallelSlot), n_slots);
+
+       /* Adjust dependency information */
+       fix_dependencies(AH);
+
+       /*
+        * Do all the early stuff in a single connection in the parent.
+        * There's no great point in running it in parallel, in fact it will
+        * actually run faster in a single connection because we avoid all the
+        * connection and setup overhead.
+        */
+       while ((next_work_item = get_next_work_item(AH, &first_unprocessed,
+                                                                                               NULL, 0)) != NULL)
+       {
+               if (next_work_item->section == SECTION_DATA ||
+                       next_work_item->section == SECTION_POST_DATA)
+                       break;
+
+               ahlog(AH, 1, "processing item %d %s %s\n",
+                         next_work_item->dumpId,
+                         next_work_item->desc, next_work_item->tag);
+
+               (void) restore_toc_entry(AH, next_work_item, ropt, false);
+
+               next_work_item->restored = true;
+               reduce_dependencies(AH, next_work_item);
+       }
+
+       /*
+        * Now close parent connection in prep for parallel steps.  We do this
+        * mainly to ensure that we don't exceed the specified number of parallel
+        * connections.
+        */
+       PQfinish(AH->connection);
+       AH->connection = NULL;
+
+       /* blow away any transient state from the old connection */
+       if (AH->currUser)
+               free(AH->currUser);
+       AH->currUser = NULL;
+       if (AH->currSchema)
+               free(AH->currSchema);
+       AH->currSchema = NULL;
+       if (AH->currTablespace)
+               free(AH->currTablespace);
+       AH->currTablespace = NULL;
+       AH->currWithOids = -1;
+
+       /*
+        * main parent loop
+        *
+        * Keep going until there is no worker still running AND there is no work
+        * left to be done.
+        */
+
+       ahlog(AH,1,"entering main parallel loop\n");
+
+       while ((next_work_item = get_next_work_item(AH, &first_unprocessed,
+                                                                                               slots, n_slots)) != NULL ||
+                  work_in_progress(slots, n_slots))
+       {
+               if (next_work_item != NULL)
+               {
+                       teReqs    reqs;
+
+                       /* If not to be dumped, don't waste time launching a worker */
+                       reqs = _tocEntryRequired(next_work_item, AH->ropt, false);
+                       if ((reqs & (REQ_SCHEMA | REQ_DATA)) == 0)
+                       {
+                               ahlog(AH, 1, "skipping item %d %s %s\n",
+                                         next_work_item->dumpId,
+                                         next_work_item->desc, next_work_item->tag);
+
+                               next_work_item->restored = true;
+                               reduce_dependencies(AH, next_work_item);
+
+                               continue;
+                       }
+
+                       if ((next_slot = get_next_slot(slots, n_slots)) != NO_SLOT)
+                       {
+                               /* There is work still to do and a worker slot available */
+                               thandle child;
+                               RestoreArgs *args;
+
+                               ahlog(AH, 1, "launching item %d %s %s\n",
+                                         next_work_item->dumpId,
+                                         next_work_item->desc, next_work_item->tag);
+
+                               next_work_item->restored = true;
+
+                               /* this memory is dealloced in mark_work_done() */
+                               args = malloc(sizeof(RestoreArgs));
+                               args->AH = CloneArchive(AH);
+                               args->te = next_work_item;
+
+                               /* run the step in a worker child */
+                               child = spawn_restore(args);
+
+                               slots[next_slot].child_id = child;
+                               slots[next_slot].args = args;
+
+                               continue;
+                       }
+               }
+
+               /*
+                * If we get here there must be work being done.  Either there is no
+                * work available to schedule (and work_in_progress returned true) or
+                * there are no slots available.  So we wait for a worker to finish,
+                * and process the result.
+                */
+               ret_child = reap_child(slots, n_slots, &work_status);
+
+               if (WIFEXITED(work_status))
+               {
+                       mark_work_done(AH, ret_child, WEXITSTATUS(work_status),
+                                                  slots, n_slots);
+               }
+               else
+               {
+                       die_horribly(AH, modulename, "worker process crashed: status %d\n",
+                                                work_status);
+               }
+       }
+
+       ahlog(AH,1,"finished main parallel loop\n");
+
+       /*
+        * Now reconnect the single parent connection.
+        */
+       ConnectDatabase((Archive *) AH, ropt->dbname,
+                                       ropt->pghost, ropt->pgport, ropt->username,
+                                       ropt->requirePassword);
+
+       _doSetFixedOutputState(AH);
+
+       /*
+        * Make sure there is no non-ACL work left due to, say,
+        * circular dependencies, or some other pathological condition.
+        * If so, do it in the single parent connection.
+        */
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       {
+               if (!te->restored)
+               {
+                       ahlog(AH, 1, "processing missed item %d %s %s\n",
+                                 te->dumpId, te->desc, te->tag);
+                       (void) restore_toc_entry(AH, te, ropt, false);
+               }
+       }
+
+       /* The ACLs will be handled back in RestoreArchive. */
+}
+
+/*
+ * create a worker child to perform a restore step in parallel
+ */
+static thandle
+spawn_restore(RestoreArgs *args)
+{
+       thandle child;
+
+       /* Ensure stdio state is quiesced before forking */
+       fflush(NULL);
+
+#ifndef WIN32
+       child = fork();
+       if (child == 0)
+       {
+               /* in child process */
+               parallel_restore(args);
+               die_horribly(args->AH, modulename,
+                                        "parallel_restore should not return\n");
+       }
+       else if (child < 0)
+       {
+               /* fork failed */
+               die_horribly(args->AH, modulename,
+                                        "could not create worker process: %s\n",
+                                        strerror(errno));
+       }
+#else
+       child = (HANDLE) _beginthreadex(NULL, 0, (void *) parallel_restore,
+                                                                       args, 0, NULL);
+       if (child == 0)
+               die_horribly(args->AH, modulename,
+                                        "could not create worker thread: %s\n",
+                                        strerror(errno));
+#endif
+
+       return child;
+}
+
+/*
+ *  collect status from a completed worker child
+ */
+static thandle
+reap_child(ParallelSlot *slots, int n_slots, int *work_status)
+{
+#ifndef WIN32
+       /* Unix is so much easier ... */
+       return wait(work_status);
+#else
+       static HANDLE *handles = NULL;
+       int hindex, snum, tnum;
+       thandle ret_child;
+       DWORD res;
+
+       /* first time around only, make space for handles to listen on */
+       if (handles == NULL)
+               handles = (HANDLE *) calloc(sizeof(HANDLE),n_slots);
+
+       /* set up list of handles to listen to */
+       for (snum=0, tnum=0; snum < n_slots; snum++)
+               if (slots[snum].child_id != 0)
+                       handles[tnum++] = slots[snum].child_id;
+
+       /* wait for one to finish */
+       hindex = WaitForMultipleObjects(tnum, handles, false, INFINITE);
+
+       /* get handle of finished thread */
+       ret_child = handles[hindex - WAIT_OBJECT_0];
+
+       /* get the result */
+       GetExitCodeThread(ret_child,&res);
+       *work_status = res;
+
+       /* dispose of handle to stop leaks */
+       CloseHandle(ret_child);
+
+       return ret_child;
+#endif
+}
+
+/*
+ * are we doing anything now?
+ */
+static bool
+work_in_progress(ParallelSlot *slots, int n_slots)
+{
+       int i;
+
+       for (i = 0; i < n_slots; i++)
+       {
+               if (slots[i].child_id != 0)
+                       return true;
+       }
+       return false;
+}
+
+/*
+ * find the first free parallel slot (if any).
+ */
+static int
+get_next_slot(ParallelSlot *slots, int n_slots)
+{
+       int i;
+
+       for (i = 0; i < n_slots; i++)
+       {
+               if (slots[i].child_id == 0)
+                       return i;
+       }
+       return NO_SLOT;
+}
+
+/*
+ * Find the next work item (if any) that is capable of being run now.
+ *
+ * To qualify, the item must have no remaining dependencies
+ * and no requirement for locks that is incompatible with
+ * items currently running.
+ *
+ * first_unprocessed is state data that tracks the location of the first
+ * TocEntry that's not marked 'restored'.  This avoids O(N^2) search time
+ * with long TOC lists.  (Even though the constant is pretty small, it'd
+ * get us eventually.)
+ *
+ * pref_non_data is for an alternative selection algorithm that gives
+ * preference to non-data items if there is already a data load running.
+ * It is currently disabled.
+ */
+static TocEntry *
+get_next_work_item(ArchiveHandle *AH, TocEntry **first_unprocessed,
+                                  ParallelSlot *slots, int n_slots)
+{
+       bool      pref_non_data = false; /* or get from AH->ropt */
+       TocEntry *data_te = NULL;
+       TocEntry *te;
+       int       i,j,k;
+
+       /*
+        * Bogus heuristics for pref_non_data
+        */
+       if (pref_non_data)
+       {
+               int count = 0;
+
+               for (k=0; k < n_slots; k++)
+                       if (slots[k].args->te != NULL &&
+                               slots[k].args->te->section == SECTION_DATA)
+                               count++;
+               if (n_slots == 0 || count * 4 < n_slots)
+                       pref_non_data = false;
+       }
+
+       /*
+        * Advance first_unprocessed if possible.
+        */
+       for (te = *first_unprocessed; te != AH->toc; te = te->next)
+       {
+               if (!te->restored)
+                       break;
+       }
+       *first_unprocessed = te;
+
+       /*
+        * Search from first_unprocessed until we find an available item.
+        */
+       for (; te != AH->toc; te = te->next)
+       {
+               bool conflicts = false;
+
+               /* Ignore if already done or still waiting on dependencies */
+               if (te->restored || te->depCount > 0)
+                       continue;
+
+               /*
+                * Check to see if the item would need exclusive lock on something
+                * that a currently running item also needs lock on.  If so, we
+                * don't want to schedule them together.
+                */
+               for (i = 0; i < n_slots && !conflicts; i++)
+               {
+                       TocEntry *running_te;
+
+                       if (slots[i].args == NULL)
+                               continue;
+                       running_te = slots[i].args->te;
+                       for (j = 0; j < te->nLockDeps && !conflicts; j++)
+                       {
+                               for (k = 0; k < running_te->nLockDeps; k++)
+                               {
+                                       if (te->lockDeps[j] == running_te->lockDeps[k])
+                                       {
+                                               conflicts = true;
+                                               break;
+                                       }
+                               }
+                       }
+               }
+
+               if (conflicts)
+                       continue;
+
+               if (pref_non_data && te->section == SECTION_DATA)
+               {
+                       if (data_te == NULL)
+                               data_te = te;
+                       continue;
+               }
+
+               /* passed all tests, so this item can run */
+               return te;
+       }
+
+       if (data_te != NULL)
+               return data_te;
+
+       ahlog(AH,2,"no item ready\n");
+       return NULL;
+}
+
+
+/*
+ * Restore a single TOC item in parallel with others
+ *
+ * this is the procedure run as a thread (Windows) or a
+ * separate process (everything else).
+ */
+static parallel_restore_result
+parallel_restore(RestoreArgs *args)
+{
+       ArchiveHandle *AH = args->AH;
+       TocEntry *te = args->te;
+       RestoreOptions *ropt = AH->ropt;
+       int retval;
+
+       /*
+        * Close and reopen the input file so we have a private file pointer
+        * that doesn't stomp on anyone else's file pointer.
+        *
+        * Note: on Windows, since we are using threads not processes, this
+        * *doesn't* close the original file pointer but just open a new one.
+        */
+       (AH->ReopenPtr) (AH);
+
+       /*
+        * We need our own database connection, too
+        */
+       ConnectDatabase((Archive *) AH, ropt->dbname,
+                                       ropt->pghost, ropt->pgport, ropt->username,
+                                       ropt->requirePassword);
+
+       _doSetFixedOutputState(AH);
+
+       /* Restore the TOC item */
+       retval = restore_toc_entry(AH, te, ropt, true);
+
+       /* And clean up */
+       PQfinish(AH->connection);
+       AH->connection = NULL;
+
+       (AH->ClosePtr) (AH);
+
+       if (retval == 0 && AH->public.n_errors)
+               retval = WORKER_IGNORED_ERRORS;
+
+#ifndef WIN32
+       exit(retval);
+#else
+       return retval;
+#endif
+}
+
+
+/*
+ * Housekeeping to be done after a step has been parallel restored.
+ *
+ * Clear the appropriate slot, free all the extra memory we allocated,
+ * update status, and reduce the dependency count of any dependent items.
+ */
+static void
+mark_work_done(ArchiveHandle *AH, thandle worker, int status,
+                          ParallelSlot *slots, int n_slots)
+{
+       TocEntry *te = NULL;
+       int i;
+
+       for (i = 0; i < n_slots; i++)
+       {
+               if (slots[i].child_id == worker)
+               {
+                       slots[i].child_id = 0;
+                       te = slots[i].args->te;
+                       DeCloneArchive(slots[i].args->AH);
+                       free(slots[i].args);
+                       slots[i].args = NULL;
+
+                       break;
+               }
+       }
+
+       if (te == NULL)
+               die_horribly(AH, modulename, "failed to find slot for finished worker\n");
+
+       ahlog(AH, 1, "finished item %d %s %s\n",
+                 te->dumpId, te->desc, te->tag);
+
+       if (status == WORKER_CREATE_DONE)
+               mark_create_done(AH, te);
+       else if (status == WORKER_INHIBIT_DATA)
+       {
+               inhibit_data_for_failed_table(AH, te);
+               AH->public.n_errors++;
+       }
+       else if (status == WORKER_IGNORED_ERRORS)
+               AH->public.n_errors++;
+       else if (status != 0)
+               die_horribly(AH, modulename, "worker process failed: exit code %d\n",
+                                        status);
+
+       reduce_dependencies(AH, te);
+}
+
+
+/*
+ * Process the dependency information into a form useful for parallel restore.
+ *
+ * We set up depCount fields that are the number of as-yet-unprocessed
+ * dependencies for each TOC entry.
+ *
+ * We also identify locking dependencies so that we can avoid trying to
+ * schedule conflicting items at the same time.
+ */
+static void
+fix_dependencies(ArchiveHandle *AH)
+{
+       TocEntry  **tocsByDumpId;
+       TocEntry   *te;
+       int i;
+
+       /*
+        * For some of the steps here, it is convenient to have an array that
+        * indexes the TOC entries by dump ID, rather than searching the TOC
+        * list repeatedly.  Entries for dump IDs not present in the TOC will
+        * be NULL.
+        *
+        * Also, initialize the depCount fields.
+        */
+       tocsByDumpId = (TocEntry **) calloc(AH->maxDumpId, sizeof(TocEntry *));
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       {
+               tocsByDumpId[te->dumpId - 1] = te;
+               te->depCount = te->nDeps;
+       }
+
+       /*
+        * POST_DATA items that are shown as depending on a table need to be
+        * re-pointed to depend on that table's data, instead.  This ensures they
+        * won't get scheduled until the data has been loaded.  We handle this by
+        * first finding TABLE/TABLE DATA pairs and then scanning all the
+        * dependencies.
+        *
+        * Note: currently, a TABLE DATA should always have exactly one
+        * dependency, on its TABLE item.  So we don't bother to search,
+        * but look just at the first dependency.  We do trouble to make sure
+        * that it's a TABLE, if possible.  However, if the dependency isn't
+        * in the archive then just assume it was a TABLE; this is to cover
+        * cases where the table was suppressed but we have the data and some
+        * dependent post-data items.
+        */
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       {
+               if (strcmp(te->desc, "TABLE DATA") == 0 && te->nDeps > 0)
+               {
+                       DumpId  tableId = te->dependencies[0];
+
+                       if (tocsByDumpId[tableId - 1] == NULL ||
+                               strcmp(tocsByDumpId[tableId - 1]->desc, "TABLE") == 0)
+                       {
+                               repoint_table_dependencies(AH, tableId, te->dumpId);
+                       }
+               }
+       }
+
+       /*
+        * Pre-8.4 versions of pg_dump neglected to set up a dependency from
+        * BLOB COMMENTS to BLOBS.  Cope.  (We assume there's only one BLOBS
+        * and only one BLOB COMMENTS in such files.)
+        */
+       if (AH->version < K_VERS_1_11)
+       {
+               for (te = AH->toc->next; te != AH->toc; te = te->next)
+               {
+                       if (strcmp(te->desc, "BLOB COMMENTS") == 0 && te->nDeps == 0)
+                       {
+                               TocEntry   *te2;
+
+                               for (te2 = AH->toc->next; te2 != AH->toc; te2 = te2->next)
+                               {
+                                       if (strcmp(te2->desc, "BLOBS") == 0)
+                                       {
+                                               te->dependencies = (DumpId *) malloc(sizeof(DumpId));
+                                               te->dependencies[0] = te2->dumpId;
+                                               te->nDeps++;
+                                               te->depCount++;
+                                               break;
+                                       }
+                               }
+                               break;
+                       }
+               }
+       }
+
+       /*
+        * It is possible that the dependencies list items that are not in the
+        * archive at all.  Subtract such items from the depCounts.
+        */
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       {
+               for (i = 0; i < te->nDeps; i++)
+               {
+                       if (tocsByDumpId[te->dependencies[i] - 1] == NULL)
+                               te->depCount--;
+               }
+       }
+
+       /*
+        * Lastly, work out the locking dependencies.
+        */
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       {
+               te->lockDeps = NULL;
+               te->nLockDeps = 0;
+               identify_locking_dependencies(te, tocsByDumpId);
+       }
+
+       free(tocsByDumpId);
+}
+
+/*
+ * Change dependencies on tableId to depend on tableDataId instead,
+ * but only in POST_DATA items.
+ */
+static void
+repoint_table_dependencies(ArchiveHandle *AH,
+                                                  DumpId tableId, DumpId tableDataId)
+{
+       TocEntry   *te;
+       int i;
+
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       {
+               if (te->section != SECTION_POST_DATA)
+                       continue;
+               for (i = 0; i < te->nDeps; i++)
+               {
+                       if (te->dependencies[i] == tableId)
+                       {
+                               te->dependencies[i] = tableDataId;
+                               ahlog(AH, 2, "transferring dependency %d -> %d to %d\n",
+                                         te->dumpId, tableId, tableDataId);
+                       }
+               }
+       }
+}
+
+/*
+ * Identify which objects we'll need exclusive lock on in order to restore
+ * the given TOC entry (*other* than the one identified by the TOC entry
+ * itself).  Record their dump IDs in the entry's lockDeps[] array.
+ * tocsByDumpId[] is a convenience array to avoid searching the TOC
+ * for each dependency.
+ */
+static void
+identify_locking_dependencies(TocEntry *te, TocEntry **tocsByDumpId)
+{
+       DumpId     *lockids;
+       int                     nlockids;
+       int                     i;
+
+       /* Quick exit if no dependencies at all */
+       if (te->nDeps == 0)
+               return;
+
+       /* Exit if this entry doesn't need exclusive lock on other objects */
+       if (!(strcmp(te->desc, "CONSTRAINT") == 0 ||
+                 strcmp(te->desc, "CHECK CONSTRAINT") == 0 ||
+                 strcmp(te->desc, "FK CONSTRAINT") == 0 ||
+                 strcmp(te->desc, "RULE") == 0 ||
+                 strcmp(te->desc, "TRIGGER") == 0))
+               return;
+
+       /*
+        * We assume the item requires exclusive lock on each TABLE item
+        * listed among its dependencies.
+        */
+       lockids = (DumpId *) malloc(te->nDeps * sizeof(DumpId));
+       nlockids = 0;
+       for (i = 0; i < te->nDeps; i++)
+       {
+               DumpId  depid = te->dependencies[i];
+
+               if (tocsByDumpId[depid - 1] &&
+                       strcmp(tocsByDumpId[depid - 1]->desc, "TABLE") == 0)
+                       lockids[nlockids++] = depid;
+       }
+
+       if (nlockids == 0)
+       {
+               free(lockids);
+               return;
+       }
+
+       te->lockDeps = realloc(lockids, nlockids * sizeof(DumpId));
+       te->nLockDeps = nlockids;
+}
+
+/*
+ * Remove the specified TOC entry from the depCounts of items that depend on
+ * it, thereby possibly making them ready-to-run.
+ */
+static void
+reduce_dependencies(ArchiveHandle *AH, TocEntry *te)
+{
+       DumpId target = te->dumpId;
+       int i;
+
+       ahlog(AH,2,"reducing dependencies for %d\n",target);
+
+       /*
+        * We must examine all entries, not only the ones after the target item,
+        * because if the user used a -L switch then the original dependency-
+        * respecting order has been destroyed by SortTocFromFile.
+        */
+       for (te = AH->toc->next; te != AH->toc; te = te->next)
+       {
+               for (i = 0; i < te->nDeps; i++)
+               {
+                       if (te->dependencies[i] == target)
+                               te->depCount--;
+               }
+       }
+}
+
+/*
+ * Set the created flag on the DATA member corresponding to the given
+ * TABLE member
+ */
+static void
+mark_create_done(ArchiveHandle *AH, TocEntry *te)
+{
+       TocEntry   *tes;
+
+       for (tes = AH->toc->next; tes != AH->toc; tes = tes->next)
+       {
+               if (strcmp(tes->desc, "TABLE DATA") == 0 &&
+                       strcmp(tes->tag, te->tag) == 0 &&
+                       strcmp(tes->namespace ? tes->namespace : "",
+                                  te->namespace ? te->namespace : "") == 0)
+               {
+                       tes->created = true;
+                       break;
+               }
+       }
+}
+
+/*
+ * Mark the DATA member corresponding to the given TABLE member
+ * as not wanted
+ */
+static void
+inhibit_data_for_failed_table(ArchiveHandle *AH, TocEntry *te)
+{
+       RestoreOptions *ropt = AH->ropt;
+       TocEntry   *tes;
+
+       ahlog(AH, 1, "table \"%s\" could not be created, will not restore its data\n",
+                 te->tag);
+
+       for (tes = AH->toc->next; tes != AH->toc; tes = tes->next)
+       {
+               if (strcmp(tes->desc, "TABLE DATA") == 0 &&
+                       strcmp(tes->tag, te->tag) == 0 &&
+                       strcmp(tes->namespace ? tes->namespace : "",
+                                  te->namespace ? te->namespace : "") == 0)
+               {
+                       /* mark it unwanted; we assume idWanted array already exists */
+                       ropt->idWanted[tes->dumpId - 1] = false;
+                       break;
+               }
+       }
+}
+
+
+/*
+ * Clone and de-clone routines used in parallel restoration.
+ *
+ * Enough of the structure is cloned to ensure that there is no
+ * conflict between different threads each with their own clone.
+ *
+ * These could be public, but no need at present.
+ */
+static ArchiveHandle *
+CloneArchive(ArchiveHandle *AH)
+{
+       ArchiveHandle *clone;
+
+       /* Make a "flat" copy */
+       clone = (ArchiveHandle *) malloc(sizeof(ArchiveHandle)); 
+       if (clone == NULL)
+               die_horribly(AH, modulename, "out of memory\n");
+       memcpy(clone, AH, sizeof(ArchiveHandle));
+
+       /* Handle format-independent fields */
+       clone->pgCopyBuf = createPQExpBuffer();
+       clone->sqlBuf = createPQExpBuffer();
+       clone->sqlparse.tagBuf = NULL;
+
+       /* The clone will have its own connection, so disregard connection state */
+       clone->connection = NULL;
+       clone->currUser = NULL;
+       clone->currSchema = NULL;
+       clone->currTablespace = NULL;
+       clone->currWithOids = -1;
+
+       /* savedPassword must be local in case we change it while connecting */
+       if (clone->savedPassword)
+               clone->savedPassword = strdup(clone->savedPassword);
+
+       /* clone has its own error count, too */
+       clone->public.n_errors = 0;
+
+       /* Let the format-specific code have a chance too */
+       (clone->ClonePtr) (clone);
+
+       return clone;
+}
+
+/*
+ * Release clone-local storage.
+ *
+ * Note: we assume any clone-local connection was already closed.
+ */
+static void
+DeCloneArchive(ArchiveHandle *AH)
+{
+       /* Clear format-specific state */
+       (AH->DeClonePtr) (AH);
+
+       /* Clear state allocated by CloneArchive */
+       destroyPQExpBuffer(AH->pgCopyBuf);
+       destroyPQExpBuffer(AH->sqlBuf);
+       if (AH->sqlparse.tagBuf)
+               destroyPQExpBuffer(AH->sqlparse.tagBuf);
+
+       /* Clear any connection-local state */
+       if (AH->currUser)
+               free(AH->currUser);
+       if (AH->currSchema)
+               free(AH->currSchema);
+       if (AH->currTablespace)
+               free(AH->currTablespace);
+       if (AH->savedPassword)
+               free(AH->savedPassword);
+
+       free(AH);
+}
index d8520ec9d6bb15485c0cbb78549a373a42ee0ca5..43756eba4d926753a764e22e3ee391e50c595ca8 100644 (file)
@@ -17,7 +17,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_archiver.h,v 1.76 2007/11/07 12:24:24 petere Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_archiver.h,v 1.77 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -62,7 +62,7 @@ typedef z_stream *z_streamp;
 #endif
 
 #define K_VERS_MAJOR 1
-#define K_VERS_MINOR 10
+#define K_VERS_MINOR 11
 #define K_VERS_REV 0
 
 /* Data block types */
@@ -85,8 +85,9 @@ typedef z_stream *z_streamp;
 #define K_VERS_1_9 (( (1 * 256 + 9) * 256 + 0) * 256 + 0)              /* add default_with_oids
                                                                                                                                 * tracking */
 #define K_VERS_1_10 (( (1 * 256 + 10) * 256 + 0) * 256 + 0)            /* add tablespace */
+#define K_VERS_1_11 (( (1 * 256 + 11) * 256 + 0) * 256 + 0)            /* add toc section indicator */
 
-#define K_VERS_MAX (( (1 * 256 + 10) * 256 + 255) * 256 + 0)
+#define K_VERS_MAX (( (1 * 256 + 11) * 256 + 255) * 256 + 0)
 
 
 /* Flags to indicate disposition of offsets stored in files */
@@ -99,6 +100,7 @@ struct _tocEntry;
 struct _restoreList;
 
 typedef void (*ClosePtr) (struct _archiveHandle * AH);
+typedef void (*ReopenPtr) (struct _archiveHandle * AH);
 typedef void (*ArchiveEntryPtr) (struct _archiveHandle * AH, struct _tocEntry * te);
 
 typedef void (*StartDataPtr) (struct _archiveHandle * AH, struct _tocEntry * te);
@@ -120,6 +122,9 @@ typedef void (*ReadExtraTocPtr) (struct _archiveHandle * AH, struct _tocEntry *
 typedef void (*PrintExtraTocPtr) (struct _archiveHandle * AH, struct _tocEntry * te);
 typedef void (*PrintTocDataPtr) (struct _archiveHandle * AH, struct _tocEntry * te, RestoreOptions *ropt);
 
+typedef void (*ClonePtr) (struct _archiveHandle * AH);
+typedef void (*DeClonePtr) (struct _archiveHandle * AH);
+
 typedef size_t (*CustomOutPtr) (struct _archiveHandle * AH, const void *buf, size_t len);
 
 typedef struct _outputContext
@@ -212,6 +217,7 @@ typedef struct _archiveHandle
        WriteBufPtr WriteBufPtr;        /* Write a buffer of output to the archive */
        ReadBufPtr ReadBufPtr;          /* Read a buffer of input from the archive */
        ClosePtr ClosePtr;                      /* Close the archive */
+       ReopenPtr ReopenPtr;            /* Reopen the archive */
        WriteExtraTocPtr WriteExtraTocPtr;      /* Write extra TOC entry data
                                                                                 * associated with the current archive
                                                                                 * format */
@@ -225,11 +231,15 @@ typedef struct _archiveHandle
        StartBlobPtr StartBlobPtr;
        EndBlobPtr EndBlobPtr;
 
+       ClonePtr        ClonePtr;               /* Clone format-specific fields */
+       DeClonePtr      DeClonePtr;             /* Clean up cloned fields */
+
        CustomOutPtr CustomOutPtr;      /* Alternative script output routine */
 
        /* Stuff for direct DB connection */
        char       *archdbname;         /* DB name *read* from archive */
        bool            requirePassword;
+       char       *savedPassword;      /* password for ropt->username, if known */
        PGconn     *connection;
        int                     connectToDB;    /* Flag to indicate if direct DB connection is
                                                                 * required */
@@ -260,9 +270,9 @@ typedef struct _archiveHandle
                                                                 * etc */
 
        /* these vars track state to avoid sending redundant SET commands */
-       char       *currUser;           /* current username */
-       char       *currSchema;         /* current schema */
-       char       *currTablespace; /* current tablespace */
+       char       *currUser;           /* current username, or NULL if unknown */
+       char       *currSchema;         /* current schema, or NULL */
+       char       *currTablespace; /* current tablespace, or NULL */
        bool            currWithOids;   /* current default_with_oids setting */
 
        void       *lo_buf;
@@ -282,6 +292,7 @@ typedef struct _tocEntry
        struct _tocEntry *next;
        CatalogId       catalogId;
        DumpId          dumpId;
+       teSection   section;
        bool            hadDumper;              /* Archiver was passed a dumper routine (used
                                                                 * in restore) */
        char       *tag;                        /* index tag */
@@ -300,6 +311,13 @@ typedef struct _tocEntry
        DataDumperPtr dataDumper;       /* Routine to dump data for object */
        void       *dataDumperArg;      /* Arg for above routine */
        void       *formatData;         /* TOC Entry data specific to file format */
+
+       /* working state (needed only for parallel restore) */
+       bool            restored;               /* item is in progress or done */
+       bool            created;                /* set for DATA member if TABLE was created */
+       int                     depCount;               /* number of dependencies not yet restored */
+       DumpId     *lockDeps;           /* dumpIds of objects this one needs lock on */
+       int                     nLockDeps;              /* number of such dependencies */
 } TocEntry;
 
 /* Used everywhere */
index 5791ec7812816f02eb8e2f2e51a05e6f176a8d57..ebe2d6ce181cac3b7793971edd071ee1bd85937a 100644 (file)
@@ -19,7 +19,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_custom.c,v 1.40 2007/10/28 21:55:52 tgl Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_custom.c,v 1.41 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -40,6 +40,7 @@ static int    _ReadByte(ArchiveHandle *);
 static size_t _WriteBuf(ArchiveHandle *AH, const void *buf, size_t len);
 static size_t _ReadBuf(ArchiveHandle *AH, void *buf, size_t len);
 static void _CloseArchive(ArchiveHandle *AH);
+static void _ReopenArchive(ArchiveHandle *AH);
 static void _PrintTocData(ArchiveHandle *AH, TocEntry *te, RestoreOptions *ropt);
 static void _WriteExtraToc(ArchiveHandle *AH, TocEntry *te);
 static void _ReadExtraToc(ArchiveHandle *AH, TocEntry *te);
@@ -54,6 +55,8 @@ static void _StartBlob(ArchiveHandle *AH, TocEntry *te, Oid oid);
 static void _EndBlob(ArchiveHandle *AH, TocEntry *te, Oid oid);
 static void _EndBlobs(ArchiveHandle *AH, TocEntry *te);
 static void _LoadBlobs(ArchiveHandle *AH);
+static void _Clone(ArchiveHandle *AH);
+static void _DeClone(ArchiveHandle *AH);
 
 /*------------
  * Buffers used in zlib compression and extra data stored in archive and
@@ -120,6 +123,7 @@ InitArchiveFmt_Custom(ArchiveHandle *AH)
        AH->WriteBufPtr = _WriteBuf;
        AH->ReadBufPtr = _ReadBuf;
        AH->ClosePtr = _CloseArchive;
+       AH->ReopenPtr = _ReopenArchive;
        AH->PrintTocDataPtr = _PrintTocData;
        AH->ReadExtraTocPtr = _ReadExtraToc;
        AH->WriteExtraTocPtr = _WriteExtraToc;
@@ -129,6 +133,8 @@ InitArchiveFmt_Custom(ArchiveHandle *AH)
        AH->StartBlobPtr = _StartBlob;
        AH->EndBlobPtr = _EndBlob;
        AH->EndBlobsPtr = _EndBlobs;
+       AH->ClonePtr = _Clone;  
+       AH->DeClonePtr = _DeClone;      
 
        /*
         * Set up some special context used in compressing data.
@@ -569,7 +575,6 @@ _PrintData(ArchiveHandle *AH)
                zp->avail_in = blkLen;
 
 #ifdef HAVE_LIBZ
-
                if (AH->compression != 0)
                {
                        while (zp->avail_in != 0)
@@ -585,15 +590,12 @@ _PrintData(ArchiveHandle *AH)
                        }
                }
                else
-               {
 #endif
+               {
                        in[zp->avail_in] = '\0';
                        ahwrite(in, 1, zp->avail_in, AH);
                        zp->avail_in = 0;
-
-#ifdef HAVE_LIBZ
                }
-#endif
                blkLen = ReadInt(AH);
        }
 
@@ -822,11 +824,9 @@ _CloseArchive(ArchiveHandle *AH)
                 * expect to be doing seeks to read the data back - it may be ok to
                 * just use the existing self-consistent block formatting.
                 */
-               if (ctx->hasSeek)
-               {
-                       fseeko(AH->FH, tpos, SEEK_SET);
+               if (ctx->hasSeek &&
+                       fseeko(AH->FH, tpos, SEEK_SET) == 0)
                        WriteToc(AH);
-               }
        }
 
        if (fclose(AH->FH) != 0)
@@ -835,6 +835,48 @@ _CloseArchive(ArchiveHandle *AH)
        AH->FH = NULL;
 }
 
+/*
+ * Reopen the archive's file handle.
+ *
+ * We close the original file handle, except on Windows.  (The difference
+ * is because on Windows, this is used within a multithreading context,
+ * and we don't want a thread closing the parent file handle.)
+ */
+static void 
+_ReopenArchive(ArchiveHandle *AH)
+{
+       lclContext *ctx = (lclContext *) AH->formatData;
+       pgoff_t         tpos;
+
+       if (AH->mode == archModeWrite)
+               die_horribly(AH,modulename,"can only reopen input archives\n");
+       if (AH->fSpec == NULL || strcmp(AH->fSpec, "") == 0)
+               die_horribly(AH,modulename,"cannot reopen stdin\n");
+       if (!ctx->hasSeek)
+               die_horribly(AH,modulename,"cannot reopen non-seekable file\n");
+
+       errno = 0;
+       tpos = ftello(AH->FH);
+       if (errno)
+               die_horribly(AH, modulename, "could not determine seek position in archive file: %s\n", 
+                                        strerror(errno));
+
+#ifndef WIN32
+       if (fclose(AH->FH) != 0)
+               die_horribly(AH, modulename, "could not close archive file: %s\n", 
+                                        strerror(errno));
+#endif
+
+       AH->FH = fopen(AH->fSpec, PG_BINARY_R);
+       if (!AH->FH)
+               die_horribly(AH, modulename, "could not open input file \"%s\": %s\n",
+                                        AH->fSpec, strerror(errno));
+
+       if (fseeko(AH->FH, tpos, SEEK_SET) != 0)
+               die_horribly(AH, modulename, "could not set seek position in archive file: %s\n", 
+                                        strerror(errno));
+}
+
 /*--------------------------------------------------
  * END OF FORMAT CALLBACKS
  *--------------------------------------------------
@@ -990,7 +1032,6 @@ _DoDeflate(ArchiveHandle *AH, lclContext *ctx, int flush)
 /*
  * Terminate zlib context and flush it's buffers. If no zlib
  * then just return.
- *
  */
 static void
 _EndDataCompressor(ArchiveHandle *AH, TocEntry *te)
@@ -1020,3 +1061,44 @@ _EndDataCompressor(ArchiveHandle *AH, TocEntry *te)
        /* Send the end marker */
        WriteInt(AH, 0);
 }
+
+
+/*
+ * Clone format-specific fields during parallel restoration.
+ */
+static void
+_Clone(ArchiveHandle *AH)
+{
+       lclContext *ctx = (lclContext *) AH->formatData;
+
+       AH->formatData = (lclContext *) malloc(sizeof(lclContext));
+       if (AH->formatData == NULL)
+               die_horribly(AH, modulename, "out of memory\n");
+       memcpy(AH->formatData, ctx, sizeof(lclContext));
+       ctx = (lclContext *) AH->formatData;
+
+       ctx->zp = (z_streamp) malloc(sizeof(z_stream));
+       ctx->zlibOut = (char *) malloc(zlibOutSize + 1);
+       ctx->zlibIn = (char *) malloc(ctx->inSize);
+
+       if (ctx->zp == NULL || ctx->zlibOut == NULL || ctx->zlibIn == NULL)
+               die_horribly(AH, modulename, "out of memory\n");
+
+       /*
+        * Note: we do not make a local lo_buf because we expect at most one
+        * BLOBS entry per archive, so no parallelism is possible.  Likewise,
+        * TOC-entry-local state isn't an issue because any one TOC entry is
+        * touched by just one worker child.
+        */
+}
+
+static void
+_DeClone(ArchiveHandle *AH)
+{
+       lclContext *ctx = (lclContext *) AH->formatData;
+       
+       free(ctx->zlibOut);
+       free(ctx->zlibIn);
+       free(ctx->zp);
+       free(ctx);
+}
index 0818b353e4312f69c8476caaa82b20c1363dfcc6..89fbabdfe6db4acb6134c984b0a21056762338c1 100644 (file)
@@ -5,7 +5,7 @@
  *     Implements the basic DB functions used by the archiver.
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_db.c,v 1.80 2008/08/16 02:25:06 tgl Exp $
+ *       $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_db.c,v 1.81 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -116,29 +116,36 @@ ReconnectToServer(ArchiveHandle *AH, const char *dbname, const char *username)
 
 /*
  * Connect to the db again.
+ *
+ * Note: it's not really all that sensible to use a single-entry password
+ * cache if the username keeps changing.  In current usage, however, the
+ * username never does change, so one savedPassword is sufficient.  We do
+ * update the cache on the off chance that the password has changed since the
+ * start of the run.
  */
 static PGconn *
 _connectDB(ArchiveHandle *AH, const char *reqdb, const char *requser)
 {
        PGconn     *newConn;
-       char       *newdb;
-       char       *newuser;
-       char       *password = NULL;
+       const char *newdb;
+       const char *newuser;
+       char       *password = AH->savedPassword;
        bool            new_pass;
 
        if (!reqdb)
                newdb = PQdb(AH->connection);
        else
-               newdb = (char *) reqdb;
+               newdb = reqdb;
 
-       if (!requser || (strlen(requser) == 0))
+       if (!requser || strlen(requser) == 0)
                newuser = PQuser(AH->connection);
        else
-               newuser = (char *) requser;
+               newuser = requser;
 
-       ahlog(AH, 1, "connecting to database \"%s\" as user \"%s\"\n", newdb, newuser);
+       ahlog(AH, 1, "connecting to database \"%s\" as user \"%s\"\n",
+                 newdb, newuser);
 
-       if (AH->requirePassword)
+       if (AH->requirePassword && password == NULL)
        {
                password = simple_prompt("Password: ", 100, false);
                if (password == NULL)
@@ -170,12 +177,13 @@ _connectDB(ArchiveHandle *AH, const char *reqdb, const char *requser)
                        if (password)
                                free(password);
                        password = simple_prompt("Password: ", 100, false);
+                       if (password == NULL)
+                               die_horribly(AH, modulename, "out of memory\n");
                        new_pass = true;
                }
        } while (new_pass);
 
-       if (password)
-               free(password);
+       AH->savedPassword = password;
 
        /* check for version mismatch */
        _check_database_version(AH);
@@ -190,6 +198,10 @@ _connectDB(ArchiveHandle *AH, const char *reqdb, const char *requser)
  * Make a database connection with the given parameters.  The
  * connection handle is returned, the parameters are stored in AHX.
  * An interactive password prompt is automatically issued if required.
+ *
+ * Note: it's not really all that sensible to use a single-entry password
+ * cache if the username keeps changing.  In current usage, however, the
+ * username never does change, so one savedPassword is sufficient.
  */
 PGconn *
 ConnectDatabase(Archive *AHX,
@@ -200,21 +212,19 @@ ConnectDatabase(Archive *AHX,
                                int reqPwd)
 {
        ArchiveHandle *AH = (ArchiveHandle *) AHX;
-       char       *password = NULL;
+       char       *password = AH->savedPassword;
        bool            new_pass;
 
        if (AH->connection)
                die_horribly(AH, modulename, "already connected to a database\n");
 
-       if (reqPwd)
+       if (reqPwd && password == NULL)
        {
                password = simple_prompt("Password: ", 100, false);
                if (password == NULL)
                        die_horribly(AH, modulename, "out of memory\n");
-               AH->requirePassword = true;
        }
-       else
-               AH->requirePassword = false;
+       AH->requirePassword = reqPwd;
 
        /*
         * Start the connection.  Loop until we have a password if requested by
@@ -236,12 +246,13 @@ ConnectDatabase(Archive *AHX,
                {
                        PQfinish(AH->connection);
                        password = simple_prompt("Password: ", 100, false);
+                       if (password == NULL)
+                               die_horribly(AH, modulename, "out of memory\n");
                        new_pass = true;
                }
        } while (new_pass);
 
-       if (password)
-               free(password);
+       AH->savedPassword = password;
 
        /* check to see that the backend connection was successfully made */
        if (PQstatus(AH->connection) == CONNECTION_BAD)
index 47a4ddd77cb3d23e11999ca904db861f8a2e2e0d..544e48722b413df3a9f694f3422a6684322303b3 100644 (file)
@@ -20,7 +20,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_files.c,v 1.34 2007/10/28 21:55:52 tgl Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_files.c,v 1.35 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -87,6 +87,7 @@ InitArchiveFmt_Files(ArchiveHandle *AH)
        AH->WriteBufPtr = _WriteBuf;
        AH->ReadBufPtr = _ReadBuf;
        AH->ClosePtr = _CloseArchive;
+       AH->ReopenPtr = NULL;
        AH->PrintTocDataPtr = _PrintTocData;
        AH->ReadExtraTocPtr = _ReadExtraToc;
        AH->WriteExtraTocPtr = _WriteExtraToc;
@@ -96,6 +97,8 @@ InitArchiveFmt_Files(ArchiveHandle *AH)
        AH->StartBlobPtr = _StartBlob;
        AH->EndBlobPtr = _EndBlob;
        AH->EndBlobsPtr = _EndBlobs;
+       AH->ClonePtr = NULL;
+       AH->DeClonePtr = NULL;
 
        /*
         * Set up some special context used in compressing data.
index f31a173ce1fe3baea83567c73cbb945179503ff3..abff1138fc2e9a38d72a9962fe0905897f71cf64 100644 (file)
@@ -17,7 +17,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_null.c,v 1.19 2006/07/14 14:52:26 momjian Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_null.c,v 1.20 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -54,12 +54,15 @@ InitArchiveFmt_Null(ArchiveHandle *AH)
        AH->WriteBytePtr = _WriteByte;
        AH->WriteBufPtr = _WriteBuf;
        AH->ClosePtr = _CloseArchive;
+       AH->ReopenPtr = NULL;
        AH->PrintTocDataPtr = _PrintTocData;
 
        AH->StartBlobsPtr = _StartBlobs;
        AH->StartBlobPtr = _StartBlob;
        AH->EndBlobPtr = _EndBlob;
        AH->EndBlobsPtr = _EndBlobs;
+       AH->ClonePtr = NULL;
+       AH->DeClonePtr = NULL;
 
        /* Initialize LO buffering */
        AH->lo_buf_size = LOBBUFSIZE;
index ddab506bf43c1fe9615888182a8de555bf7d7ff7..0156bbf9f00e54d29da233db1fa82bac9ea2c39a 100644 (file)
@@ -16,7 +16,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_tar.c,v 1.62 2007/11/15 21:14:41 momjian Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_tar.c,v 1.63 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -143,6 +143,7 @@ InitArchiveFmt_Tar(ArchiveHandle *AH)
        AH->WriteBufPtr = _WriteBuf;
        AH->ReadBufPtr = _ReadBuf;
        AH->ClosePtr = _CloseArchive;
+       AH->ReopenPtr = NULL;
        AH->PrintTocDataPtr = _PrintTocData;
        AH->ReadExtraTocPtr = _ReadExtraToc;
        AH->WriteExtraTocPtr = _WriteExtraToc;
@@ -152,6 +153,8 @@ InitArchiveFmt_Tar(ArchiveHandle *AH)
        AH->StartBlobPtr = _StartBlob;
        AH->EndBlobPtr = _EndBlob;
        AH->EndBlobsPtr = _EndBlobs;
+       AH->ClonePtr = NULL;
+       AH->DeClonePtr = NULL;
 
        /*
         * Set up some special context used in compressing data.
@@ -1383,5 +1386,4 @@ _tarWriteHeader(TAR_MEMBER *th)
 
        if (fwrite(h, 1, 512, th->tarFH) != 512)
                die_horribly(th->AH, modulename, "could not write to output file: %s\n", strerror(errno));
-
 }
index 6f057f2a33e04bd5481564ca74644ae81a1bf6e1..5ef46ae08ed52bf782c73cb50c25305876b94903 100644 (file)
@@ -12,7 +12,7 @@
  *     by PostgreSQL
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/bin/pg_dump/pg_dump.c,v 1.518 2009/02/02 19:31:39 alvherre Exp $
+ *       $PostgreSQL: pgsql/src/bin/pg_dump/pg_dump.c,v 1.519 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -695,6 +695,7 @@ main(int argc, char **argv)
        {
                /* Add placeholders to allow correct sorting of blobs */
                DumpableObject *blobobj;
+               DumpableObject *blobcobj;
 
                blobobj = (DumpableObject *) malloc(sizeof(DumpableObject));
                blobobj->objType = DO_BLOBS;
@@ -702,11 +703,12 @@ main(int argc, char **argv)
                AssignDumpId(blobobj);
                blobobj->name = strdup("BLOBS");
 
-               blobobj = (DumpableObject *) malloc(sizeof(DumpableObject));
-               blobobj->objType = DO_BLOB_COMMENTS;
-               blobobj->catId = nilCatalogId;
-               AssignDumpId(blobobj);
-               blobobj->name = strdup("BLOB COMMENTS");
+               blobcobj = (DumpableObject *) malloc(sizeof(DumpableObject));
+               blobcobj->objType = DO_BLOB_COMMENTS;
+               blobcobj->catId = nilCatalogId;
+               AssignDumpId(blobcobj);
+               blobcobj->name = strdup("BLOB COMMENTS");
+               addObjectDependency(blobcobj, blobobj->dumpId);
        }
 
        /*
@@ -1385,11 +1387,10 @@ dumpTableData(Archive *fout, TableDataInfo *tdinfo)
        }
 
        ArchiveEntry(fout, tdinfo->dobj.catId, tdinfo->dobj.dumpId,
-                                tbinfo->dobj.name,
-                                tbinfo->dobj.namespace->dobj.name,
-                                NULL,
-                                tbinfo->rolname, false,
-                                "TABLE DATA", "", "", copyStmt,
+                                tbinfo->dobj.name, tbinfo->dobj.namespace->dobj.name,
+                                NULL, tbinfo->rolname,
+                                false, "TABLE DATA", SECTION_DATA,
+                                "", "", copyStmt,
                                 tdinfo->dobj.dependencies, tdinfo->dobj.nDeps,
                                 dumpFn, tdinfo);
 
@@ -1738,6 +1739,7 @@ dumpDatabase(Archive *AH)
                                 dba,                   /* Owner */
                                 false,                 /* with oids */
                                 "DATABASE",    /* Desc */
+                                SECTION_PRE_DATA, /* Section */
                                 creaQry->data, /* Create */
                                 delQry->data,  /* Del */
                                 NULL,                  /* Copy */
@@ -1764,7 +1766,8 @@ dumpDatabase(Archive *AH)
                        appendPQExpBuffer(dbQry, ";\n");
 
                        ArchiveEntry(AH, dbCatId, createDumpId(), datname, NULL, NULL,
-                                                dba, false, "COMMENT", dbQry->data, "", NULL,
+                                                dba, false, "COMMENT", SECTION_NONE,
+                                                dbQry->data, "", NULL,
                                                 &dbDumpId, 1, NULL, NULL);
                }
        }
@@ -1802,7 +1805,8 @@ dumpEncoding(Archive *AH)
 
        ArchiveEntry(AH, nilCatalogId, createDumpId(),
                                 "ENCODING", NULL, NULL, "",
-                                false, "ENCODING", qry->data, "", NULL,
+                                false, "ENCODING", SECTION_PRE_DATA,
+                                qry->data, "", NULL,
                                 NULL, 0,
                                 NULL, NULL);
 
@@ -1828,7 +1832,8 @@ dumpStdStrings(Archive *AH)
 
        ArchiveEntry(AH, nilCatalogId, createDumpId(),
                                 "STDSTRINGS", NULL, NULL, "",
-                                false, "STDSTRINGS", qry->data, "", NULL,
+                                false, "STDSTRINGS", SECTION_PRE_DATA,
+                                qry->data, "", NULL,
                                 NULL, 0,
                                 NULL, NULL);
 
@@ -5514,9 +5519,15 @@ dumpComment(Archive *fout, const char *target,
                appendStringLiteralAH(query, comments->descr, fout);
                appendPQExpBuffer(query, ";\n");
 
+               /*
+                * We mark comments as SECTION_NONE because they really belong
+                * in the same section as their parent, whether that is
+                * pre-data or post-data.
+                */
                ArchiveEntry(fout, nilCatalogId, createDumpId(),
-                                        target, namespace, NULL, owner, false,
-                                        "COMMENT", query->data, "", NULL,
+                                        target, namespace, NULL, owner,
+                                        false, "COMMENT", SECTION_NONE,
+                                        query->data, "", NULL,
                                         &(dumpId), 1,
                                         NULL, NULL);
 
@@ -5575,9 +5586,9 @@ dumpTableComment(Archive *fout, TableInfo *tbinfo,
                        ArchiveEntry(fout, nilCatalogId, createDumpId(),
                                                 target->data,
                                                 tbinfo->dobj.namespace->dobj.name,
-                                                NULL,
-                                                tbinfo->rolname,
-                                                false, "COMMENT", query->data, "", NULL,
+                                                NULL, tbinfo->rolname,
+                                                false, "COMMENT", SECTION_NONE,
+                                                query->data, "", NULL,
                                                 &(tbinfo->dobj.dumpId), 1,
                                                 NULL, NULL);
                }
@@ -5597,9 +5608,9 @@ dumpTableComment(Archive *fout, TableInfo *tbinfo,
                        ArchiveEntry(fout, nilCatalogId, createDumpId(),
                                                 target->data,
                                                 tbinfo->dobj.namespace->dobj.name,
-                                                NULL,
-                                                tbinfo->rolname,
-                                                false, "COMMENT", query->data, "", NULL,
+                                                NULL, tbinfo->rolname,
+                                                false, "COMMENT", SECTION_NONE,
+                                                query->data, "", NULL,
                                                 &(tbinfo->dobj.dumpId), 1,
                                                 NULL, NULL);
                }
@@ -5872,15 +5883,17 @@ dumpDumpableObject(Archive *fout, DumpableObject *dobj)
                case DO_BLOBS:
                        ArchiveEntry(fout, dobj->catId, dobj->dumpId,
                                                 dobj->name, NULL, NULL, "",
-                                                false, "BLOBS", "", "", NULL,
-                                                NULL, 0,
+                                                false, "BLOBS", SECTION_DATA,
+                                                "", "", NULL,
+                                                dobj->dependencies, dobj->nDeps,
                                                 dumpBlobs, NULL);
                        break;
                case DO_BLOB_COMMENTS:
                        ArchiveEntry(fout, dobj->catId, dobj->dumpId,
                                                 dobj->name, NULL, NULL, "",
-                                                false, "BLOB COMMENTS", "", "", NULL,
-                                                NULL, 0,
+                                                false, "BLOB COMMENTS", SECTION_DATA,
+                                                "", "", NULL,
+                                                dobj->dependencies, dobj->nDeps,
                                                 dumpBlobComments, NULL);
                        break;
        }
@@ -5918,7 +5931,8 @@ dumpNamespace(Archive *fout, NamespaceInfo *nspinfo)
                                 nspinfo->dobj.name,
                                 NULL, NULL,
                                 nspinfo->rolname,
-                                false, "SCHEMA", q->data, delq->data, NULL,
+                                false, "SCHEMA", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 nspinfo->dobj.dependencies, nspinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -6021,7 +6035,8 @@ dumpEnumType(Archive *fout, TypeInfo *tinfo)
                                 tinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 tinfo->rolname, false,
-                                "TYPE", q->data, delq->data, NULL,
+                                "TYPE", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 tinfo->dobj.dependencies, tinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -6389,7 +6404,8 @@ dumpBaseType(Archive *fout, TypeInfo *tinfo)
                                 tinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 tinfo->rolname, false,
-                                "TYPE", q->data, delq->data, NULL,
+                                "TYPE", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 tinfo->dobj.dependencies, tinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -6507,7 +6523,8 @@ dumpDomain(Archive *fout, TypeInfo *tinfo)
                                 tinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 tinfo->rolname, false,
-                                "DOMAIN", q->data, delq->data, NULL,
+                                "DOMAIN", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 tinfo->dobj.dependencies, tinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -6600,7 +6617,8 @@ dumpCompositeType(Archive *fout, TypeInfo *tinfo)
                                 tinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 tinfo->rolname, false,
-                                "TYPE", q->data, delq->data, NULL,
+                                "TYPE", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 tinfo->dobj.dependencies, tinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -6653,7 +6671,8 @@ dumpShellType(Archive *fout, ShellTypeInfo *stinfo)
                                 stinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 stinfo->baseType->rolname, false,
-                                "SHELL TYPE", q->data, "", NULL,
+                                "SHELL TYPE", SECTION_PRE_DATA,
+                                q->data, "", NULL,
                                 stinfo->dobj.dependencies, stinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -6773,7 +6792,7 @@ dumpProcLang(Archive *fout, ProcLangInfo *plang)
        ArchiveEntry(fout, plang->dobj.catId, plang->dobj.dumpId,
                                 plang->dobj.name,
                                 lanschema, NULL, plang->lanowner,
-                                false, "PROCEDURAL LANGUAGE",
+                                false, "PROCEDURAL LANGUAGE", SECTION_PRE_DATA,
                                 defqry->data, delqry->data, NULL,
                                 plang->dobj.dependencies, plang->dobj.nDeps,
                                 NULL, NULL);
@@ -7331,7 +7350,8 @@ dumpFunc(Archive *fout, FuncInfo *finfo)
                                 finfo->dobj.namespace->dobj.name,
                                 NULL,
                                 finfo->rolname, false,
-                                "FUNCTION", q->data, delqry->data, NULL,
+                                "FUNCTION", SECTION_PRE_DATA,
+                                q->data, delqry->data, NULL,
                                 finfo->dobj.dependencies, finfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -7482,7 +7502,8 @@ dumpCast(Archive *fout, CastInfo *cast)
        ArchiveEntry(fout, cast->dobj.catId, cast->dobj.dumpId,
                                 castsig->data,
                                 "pg_catalog", NULL, "",
-                                false, "CAST", defqry->data, delqry->data, NULL,
+                                false, "CAST", SECTION_PRE_DATA,
+                                defqry->data, delqry->data, NULL,
                                 cast->dobj.dependencies, cast->dobj.nDeps,
                                 NULL, NULL);
 
@@ -7723,7 +7744,8 @@ dumpOpr(Archive *fout, OprInfo *oprinfo)
                                 oprinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 oprinfo->rolname,
-                                false, "OPERATOR", q->data, delq->data, NULL,
+                                false, "OPERATOR", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 oprinfo->dobj.dependencies, oprinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -8175,7 +8197,8 @@ dumpOpclass(Archive *fout, OpclassInfo *opcinfo)
                                 opcinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 opcinfo->rolname,
-                                false, "OPERATOR CLASS", q->data, delq->data, NULL,
+                                false, "OPERATOR CLASS", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 opcinfo->dobj.dependencies, opcinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -8451,7 +8474,8 @@ dumpOpfamily(Archive *fout, OpfamilyInfo *opfinfo)
                                 opfinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 opfinfo->rolname,
-                                false, "OPERATOR FAMILY", q->data, delq->data, NULL,
+                                false, "OPERATOR FAMILY", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 opfinfo->dobj.dependencies, opfinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -8564,7 +8588,8 @@ dumpConversion(Archive *fout, ConvInfo *convinfo)
                                 convinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 convinfo->rolname,
-                                false, "CONVERSION", q->data, delq->data, NULL,
+                                false, "CONVERSION", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 convinfo->dobj.dependencies, convinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -8805,7 +8830,8 @@ dumpAgg(Archive *fout, AggInfo *agginfo)
                                 agginfo->aggfn.dobj.namespace->dobj.name,
                                 NULL,
                                 agginfo->aggfn.rolname,
-                                false, "AGGREGATE", q->data, delq->data, NULL,
+                                false, "AGGREGATE", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 agginfo->aggfn.dobj.dependencies, agginfo->aggfn.dobj.nDeps,
                                 NULL, NULL);
 
@@ -8892,7 +8918,8 @@ dumpTSParser(Archive *fout, TSParserInfo *prsinfo)
                                 prsinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 "",
-                                false, "TEXT SEARCH PARSER", q->data, delq->data, NULL,
+                                false, "TEXT SEARCH PARSER", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 prsinfo->dobj.dependencies, prsinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -8981,7 +9008,8 @@ dumpTSDictionary(Archive *fout, TSDictInfo *dictinfo)
                                 dictinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 dictinfo->rolname,
-                                false, "TEXT SEARCH DICTIONARY", q->data, delq->data, NULL,
+                                false, "TEXT SEARCH DICTIONARY", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 dictinfo->dobj.dependencies, dictinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -9040,7 +9068,8 @@ dumpTSTemplate(Archive *fout, TSTemplateInfo *tmplinfo)
                                 tmplinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 "",
-                                false, "TEXT SEARCH TEMPLATE", q->data, delq->data, NULL,
+                                false, "TEXT SEARCH TEMPLATE", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 tmplinfo->dobj.dependencies, tmplinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -9170,7 +9199,8 @@ dumpTSConfig(Archive *fout, TSConfigInfo *cfginfo)
                                 cfginfo->dobj.namespace->dobj.name,
                                 NULL,
                                 cfginfo->rolname,
-                          false, "TEXT SEARCH CONFIGURATION", q->data, delq->data, NULL,
+                                false, "TEXT SEARCH CONFIGURATION", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 cfginfo->dobj.dependencies, cfginfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -9220,7 +9250,8 @@ dumpForeignDataWrapper(Archive *fout, FdwInfo *fdwinfo)
                                 NULL,
                                 NULL,
                                 fdwinfo->rolname,
-                                false, "FOREIGN DATA WRAPPER", q->data, delq->data, NULL,
+                                false, "FOREIGN DATA WRAPPER", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 fdwinfo->dobj.dependencies, fdwinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -9298,7 +9329,8 @@ dumpForeignServer(Archive *fout, ForeignServerInfo *srvinfo)
                                 NULL,
                                 NULL,
                                 srvinfo->rolname,
-                                false, "SERVER", q->data, delq->data, NULL,
+                                false, "SERVER", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 srvinfo->dobj.dependencies, srvinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -9393,8 +9425,8 @@ dumpUserMappings(Archive *fout, const char *target,
                                         namespace,
                                         NULL,
                                         owner, false,
-                                        "USER MAPPING", q->data,
-                                        delq->data, NULL,
+                                        "USER MAPPING", SECTION_PRE_DATA,
+                                        q->data, delq->data, NULL,
                                         &dumpId, 1,
                                         NULL, NULL);
        }
@@ -9447,7 +9479,8 @@ dumpACL(Archive *fout, CatalogId objCatId, DumpId objDumpId,
                                         tag, nspname,
                                         NULL,
                                         owner ? owner : "",
-                                        false, "ACL", sql->data, "", NULL,
+                                        false, "ACL", SECTION_NONE,
+                                        sql->data, "", NULL,
                                         &(objDumpId), 1,
                                         NULL, NULL);
 
@@ -9797,7 +9830,8 @@ dumpTableSchema(Archive *fout, TableInfo *tbinfo)
                        (tbinfo->relkind == RELKIND_VIEW) ? NULL : tbinfo->reltablespace,
                                 tbinfo->rolname,
                           (strcmp(reltypename, "TABLE") == 0) ? tbinfo->hasoids : false,
-                                reltypename, q->data, delq->data, NULL,
+                                reltypename, SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 tbinfo->dobj.dependencies, tbinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -9863,7 +9897,8 @@ dumpAttrDef(Archive *fout, AttrDefInfo *adinfo)
                                 tbinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 tbinfo->rolname,
-                                false, "DEFAULT", q->data, delq->data, NULL,
+                                false, "DEFAULT", SECTION_PRE_DATA,
+                                q->data, delq->data, NULL,
                                 adinfo->dobj.dependencies, adinfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -9956,7 +9991,8 @@ dumpIndex(Archive *fout, IndxInfo *indxinfo)
                                         tbinfo->dobj.namespace->dobj.name,
                                         indxinfo->tablespace,
                                         tbinfo->rolname, false,
-                                        "INDEX", q->data, delq->data, NULL,
+                                        "INDEX", SECTION_POST_DATA,
+                                        q->data, delq->data, NULL,
                                         indxinfo->dobj.dependencies, indxinfo->dobj.nDeps,
                                         NULL, NULL);
        }
@@ -10059,7 +10095,8 @@ dumpConstraint(Archive *fout, ConstraintInfo *coninfo)
                                         tbinfo->dobj.namespace->dobj.name,
                                         indxinfo->tablespace,
                                         tbinfo->rolname, false,
-                                        "CONSTRAINT", q->data, delq->data, NULL,
+                                        "CONSTRAINT", SECTION_POST_DATA,
+                                        q->data, delq->data, NULL,
                                         coninfo->dobj.dependencies, coninfo->dobj.nDeps,
                                         NULL, NULL);
        }
@@ -10091,7 +10128,8 @@ dumpConstraint(Archive *fout, ConstraintInfo *coninfo)
                                         tbinfo->dobj.namespace->dobj.name,
                                         NULL,
                                         tbinfo->rolname, false,
-                                        "FK CONSTRAINT", q->data, delq->data, NULL,
+                                        "FK CONSTRAINT", SECTION_POST_DATA,
+                                        q->data, delq->data, NULL,
                                         coninfo->dobj.dependencies, coninfo->dobj.nDeps,
                                         NULL, NULL);
        }
@@ -10125,7 +10163,8 @@ dumpConstraint(Archive *fout, ConstraintInfo *coninfo)
                                                 tbinfo->dobj.namespace->dobj.name,
                                                 NULL,
                                                 tbinfo->rolname, false,
-                                                "CHECK CONSTRAINT", q->data, delq->data, NULL,
+                                                "CHECK CONSTRAINT", SECTION_POST_DATA,
+                                                q->data, delq->data, NULL,
                                                 coninfo->dobj.dependencies, coninfo->dobj.nDeps,
                                                 NULL, NULL);
                }
@@ -10160,7 +10199,8 @@ dumpConstraint(Archive *fout, ConstraintInfo *coninfo)
                                                 tinfo->dobj.namespace->dobj.name,
                                                 NULL,
                                                 tinfo->rolname, false,
-                                                "CHECK CONSTRAINT", q->data, delq->data, NULL,
+                                                "CHECK CONSTRAINT", SECTION_POST_DATA,
+                                                q->data, delq->data, NULL,
                                                 coninfo->dobj.dependencies, coninfo->dobj.nDeps,
                                                 NULL, NULL);
                }
@@ -10433,7 +10473,8 @@ dumpSequence(Archive *fout, TableInfo *tbinfo)
                                         tbinfo->dobj.namespace->dobj.name,
                                         NULL,
                                         tbinfo->rolname,
-                                        false, "SEQUENCE", query->data, delqry->data, NULL,
+                                        false, "SEQUENCE", SECTION_PRE_DATA,
+                                        query->data, delqry->data, NULL,
                                         tbinfo->dobj.dependencies, tbinfo->dobj.nDeps,
                                         NULL, NULL);
 
@@ -10468,7 +10509,8 @@ dumpSequence(Archive *fout, TableInfo *tbinfo)
                                                         tbinfo->dobj.namespace->dobj.name,
                                                         NULL,
                                                         tbinfo->rolname,
-                                                  false, "SEQUENCE OWNED BY", query->data, "", NULL,
+                                                        false, "SEQUENCE OWNED BY", SECTION_PRE_DATA,
+                                                        query->data, "", NULL,
                                                         &(tbinfo->dobj.dumpId), 1,
                                                         NULL, NULL);
                        }
@@ -10495,7 +10537,8 @@ dumpSequence(Archive *fout, TableInfo *tbinfo)
                                         tbinfo->dobj.namespace->dobj.name,
                                         NULL,
                                         tbinfo->rolname,
-                                        false, "SEQUENCE SET", query->data, "", NULL,
+                                        false, "SEQUENCE SET", SECTION_PRE_DATA,
+                                        query->data, "", NULL,
                                         &(tbinfo->dobj.dumpId), 1,
                                         NULL, NULL);
        }
@@ -10691,7 +10734,8 @@ dumpTrigger(Archive *fout, TriggerInfo *tginfo)
                                 tbinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 tbinfo->rolname, false,
-                                "TRIGGER", query->data, delqry->data, NULL,
+                                "TRIGGER", SECTION_POST_DATA,
+                                query->data, delqry->data, NULL,
                                 tginfo->dobj.dependencies, tginfo->dobj.nDeps,
                                 NULL, NULL);
 
@@ -10810,7 +10854,8 @@ dumpRule(Archive *fout, RuleInfo *rinfo)
                                 tbinfo->dobj.namespace->dobj.name,
                                 NULL,
                                 tbinfo->rolname, false,
-                                "RULE", cmd->data, delcmd->data, NULL,
+                                "RULE", SECTION_POST_DATA,
+                                cmd->data, delcmd->data, NULL,
                                 rinfo->dobj.dependencies, rinfo->dobj.nDeps,
                                 NULL, NULL);
 
index 1d0b3c42c20dc897b7d1fbb4c75c476bc2390321..cee5f9e6208d43621609a73d1de94d84f1d07fb6 100644 (file)
@@ -34,7 +34,7 @@
  *
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_restore.c,v 1.91 2009/01/06 17:18:11 momjian Exp $
+ *             $PostgreSQL: pgsql/src/bin/pg_dump/pg_restore.c,v 1.92 2009/02/02 20:07:37 adunstan Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -93,6 +93,7 @@ main(int argc, char **argv)
                {"ignore-version", 0, NULL, 'i'},
                {"index", 1, NULL, 'I'},
                {"list", 0, NULL, 'l'},
+               {"multi-thread", 1, NULL, 'm'},
                {"no-privileges", 0, NULL, 'x'},
                {"no-acl", 0, NULL, 'x'},
                {"no-owner", 0, NULL, 'O'},
@@ -141,7 +142,7 @@ main(int argc, char **argv)
                }
        }
 
-       while ((c = getopt_long(argc, argv, "acCd:ef:F:h:iI:lL:n:Op:P:RsS:t:T:U:vWxX:1",
+       while ((c = getopt_long(argc, argv, "acCd:ef:F:h:iI:lL:m:n:Op:P:RsS:t:T:U:vWxX:1",
                                                        cmdopts, NULL)) != -1)
        {
                switch (c)
@@ -184,6 +185,10 @@ main(int argc, char **argv)
                                opts->tocFile = strdup(optarg);
                                break;
 
+                       case 'm':                       /* number of restore threads */
+                               opts->number_of_threads = atoi(optarg);
+                               break;
+
                        case 'n':                       /* Dump data for this schema only */
                                opts->schemaNames = strdup(optarg);
                                break;
@@ -269,7 +274,10 @@ main(int argc, char **argv)
                                break;
 
                        case 0:
-                               /* This covers the long options equivalent to -X xxx. */
+                               /* 
+                                * This covers the long options without a short equivalent, 
+                                * including those equivalent to -X xxx. 
+                                */
                                break;
 
                        case 2:                         /* SET ROLE */
@@ -301,6 +309,14 @@ main(int argc, char **argv)
                opts->useDB = 1;
        }
 
+       /* Can't do single-txn mode with multiple connections */
+       if (opts->single_txn && opts->number_of_threads > 1)
+       {
+               fprintf(stderr, _("%s: cannot specify both --single-transaction and multiple threads\n"),
+                               progname);
+               exit(1);
+       }
+
        opts->disable_triggers = disable_triggers;
        opts->noDataForFailedTables = no_data_for_failed_tables;
        opts->noTablespace = outputNoTablespaces;
@@ -308,10 +324,8 @@ main(int argc, char **argv)
 
        if (opts->formatName)
        {
-
                switch (opts->formatName[0])
                {
-
                        case 'c':
                        case 'C':
                                opts->format = archCustom;
@@ -396,6 +410,7 @@ usage(const char *progname)
        printf(_("  -I, --index=NAME         restore named index\n"));
        printf(_("  -L, --use-list=FILENAME  use specified table of contents for ordering\n"
                         "                           output from this file\n"));
+       printf(_("  -m, --multi-thread=NUM   use this many parallel connections to restore\n"));
        printf(_("  -n, --schema=NAME        restore only objects in this schema\n"));
        printf(_("  -O, --no-owner           skip restoration of object ownership\n"));
        printf(_("  -P, --function=NAME(args)\n"