Fix pg_upgrade to preserve toast relfrozenxids for old 8.3 servers.
authorBruce Momjian <bruce@momjian.us>
Thu, 1 Sep 2011 01:50:00 +0000 (21:50 -0400)
committerBruce Momjian <bruce@momjian.us>
Thu, 1 Sep 2011 01:50:00 +0000 (21:50 -0400)
This fixes a pg_upgrade bug that could lead to query errors when
clog files are improperly removed.

Backpatch to 8.4, 9.0, 9.1.

src/bin/pg_dump/pg_dump.c

index d6a547fc47559dda2399a9da20947599497cd757..b73392b6258d5ec300e7c28df063e7fa975c74df 100644 (file)
@@ -3516,14 +3516,13 @@ getTables(int *numTables)
                 * owning column, if any (note this dependency is AUTO as of 8.2)
                 */
                appendPQExpBuffer(query,
-                                                 "SELECT c.tableoid, c.oid, relname, "
-                                                 "relacl, relkind, relnamespace, "
-                                                 "(%s relowner) AS rolname, "
-                                                 "relchecks, (reltriggers <> 0) AS relhastriggers, "
-                                                 "relhasindex, relhasrules, relhasoids, "
-                                                 "relfrozenxid, "
-                                                 "0 AS toid, "
-                                                 "0 AS tfrozenxid, "
+                                                 "SELECT c.tableoid, c.oid, c.relname, "
+                                                 "c.relacl, c.relkind, c.relnamespace, "
+                                                 "(%s c.relowner) AS rolname, "
+                                                 "c.relchecks, (c.reltriggers <> 0) AS relhastriggers, "
+                                                 "c.relhasindex, c.relhasrules, c.relhasoids, "
+                                                 "c.relfrozenxid, tc.oid AS toid, "
+                                                 "tc.relfrozenxid AS tfrozenxid, "
                                                  "NULL AS reloftype, "
                                                  "d.refobjid AS owning_tab, "
                                                  "d.refobjsubid AS owning_col, "
@@ -3536,7 +3535,8 @@ getTables(int *numTables)
                                                  "d.classid = c.tableoid AND d.objid = c.oid AND "
                                                  "d.objsubid = 0 AND "
                                                  "d.refclassid = c.tableoid AND d.deptype = 'a') "
-                                                 "WHERE relkind in ('%c', '%c', '%c', '%c') "
+                                          "LEFT JOIN pg_class tc ON (c.reltoastrelid = tc.oid) "
+                                                 "WHERE c.relkind in ('%c', '%c', '%c', '%c') "
                                                  "ORDER BY c.oid",
                                                  username_subquery,
                                                  RELKIND_SEQUENCE,