Revisit AlterTableCreateToastTable's API once again, hoping to make it what

pg_migrator actually needs and not just a partial solution.  We have to be
able to specify the OID that the new toast table should be created with.

diff --git a/src/backend/catalog/toasting.c b/src/backend/catalog/toasting.c
index 510c88ec33801d84cc20d92f97e448ac47a4468d..b284cd23aa9a3bc92dc16abf5f63d03cb4b62155 100644 (file)
--- a/src/backend/catalog/toasting.c
+++ b/src/backend/catalog/toasting.c
@@ -8,7 +8,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/catalog/toasting.c,v 1.16 2009/06/11 14:48:55 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/catalog/toasting.c,v 1.17 2009/06/11 20:46:11 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -43,6 +43,11 @@ static bool needs_toast_table(Relation rel);
  *     then create a toast table for it.  (With the force option, make
  *     a toast table even if it appears unnecessary.)
  *
+ * The caller can also specify the OID to be used for the toast table.
+ * Usually, toastOid should be InvalidOid to allow a free OID to be assigned.
+ * (This option, as well as the force option, is not used by core Postgres,
+ * but is provided to support pg_migrator.)
+ *
  * reloptions for the toast table can be passed, too.  Pass (Datum) 0
  * for default reloptions.
  *
@@ -51,7 +56,8 @@ static bool needs_toast_table(Relation rel);
  * to end with CommandCounterIncrement if it makes any changes.
  */
 void
-AlterTableCreateToastTable(Oid relOid, Datum reloptions, bool force)
+AlterTableCreateToastTable(Oid relOid, Oid toastOid,
+                          Datum reloptions, bool force)
 {
    Relation    rel;
 
@@ -63,7 +69,7 @@ AlterTableCreateToastTable(Oid relOid, Datum reloptions, bool force)
    rel = heap_open(relOid, AccessExclusiveLock);
 
    /* create_toast_table does all the work */
-   (void) create_toast_table(rel, InvalidOid, InvalidOid, reloptions, force);
+   (void) create_toast_table(rel, toastOid, InvalidOid, reloptions, force);
 
    heap_close(rel, NoLock);
 }
@@ -101,8 +107,8 @@ BootstrapToastTable(char *relName, Oid toastOid, Oid toastIndexOid)
  * create_toast_table --- internal workhorse
  *
  * rel is already opened and exclusive-locked
- * toastOid and toastIndexOid are normally InvalidOid, but during
- * bootstrap they can be nonzero to specify hand-assigned OIDs
+ * toastOid and toastIndexOid are normally InvalidOid, but
+ * either or both can be nonzero to specify caller-assigned OIDs
  */
 static bool
 create_toast_table(Relation rel, Oid toastOid, Oid toastIndexOid,

diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c
index ca9999d5f4ac3f9e2c5c7140b984dcc0a5227172..1fabfb3d65c9f27356718c1411d7d5abe162f979 100644 (file)
--- a/src/backend/commands/cluster.c
+++ b/src/backend/commands/cluster.c
@@ -11,7 +11,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/commands/cluster.c,v 1.185 2009/06/11 14:48:55 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/commands/cluster.c,v 1.186 2009/06/11 20:46:11 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -741,7 +741,7 @@ make_new_heap(Oid OIDOldHeap, const char *NewName, Oid NewTableSpace)
        if (isNull)
            reloptions = (Datum) 0;
    }
-   AlterTableCreateToastTable(OIDNewHeap, reloptions, false);
+   AlterTableCreateToastTable(OIDNewHeap, InvalidOid, reloptions, false);
 
    if (OidIsValid(toastid))
        ReleaseSysCache(tuple);

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 56bf67eef4cfe2fa3d064ba1bb6da648a5398ec5..f75261160e7335c47ddc7227a5ac0548eb00f217 100644 (file)
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/commands/tablecmds.c,v 1.286 2009/06/11 14:48:56 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/commands/tablecmds.c,v 1.287 2009/06/11 20:46:11 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2585,7 +2585,8 @@ ATRewriteCatalogs(List **wqueue)
            (tab->subcmds[AT_PASS_ADD_COL] ||
             tab->subcmds[AT_PASS_ALTER_TYPE] ||
             tab->subcmds[AT_PASS_COL_ATTRS]))
-           AlterTableCreateToastTable(tab->relid, (Datum) 0, false);
+           AlterTableCreateToastTable(tab->relid, InvalidOid,
+                                      (Datum) 0, false);
    }
 }
 

diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
index 1c6fd02e1fec14518c9c37aa5af13869749148ea..1fddf10bc9a353002a666767e94153d35de9d001 100644 (file)
--- a/src/backend/executor/execMain.c
+++ b/src/backend/executor/execMain.c
@@ -26,7 +26,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/executor/execMain.c,v 1.325 2009/06/11 14:48:56 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/executor/execMain.c,v 1.326 2009/06/11 20:46:11 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2953,7 +2953,7 @@ OpenIntoRel(QueryDesc *queryDesc)
 
    (void) heap_reloptions(RELKIND_TOASTVALUE, reloptions, true);
 
-   AlterTableCreateToastTable(intoRelationId, reloptions, false);
+   AlterTableCreateToastTable(intoRelationId, InvalidOid, reloptions, false);
 
    /*
     * And open the constructed table for writing.

diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c
index 84b357665dd19d9edbbb1226370a568fb8b6d0d2..f51f90f86b4a296b3c43a7126f5a0052c3c276b9 100644 (file)
--- a/src/backend/tcop/utility.c
+++ b/src/backend/tcop/utility.c
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/tcop/utility.c,v 1.308 2009/06/11 14:49:02 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/tcop/utility.c,v 1.309 2009/06/11 20:46:11 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -447,6 +447,7 @@ ProcessUtility(Node *parsetree,
                                               true);
 
                        AlterTableCreateToastTable(relOid,
+                                                  InvalidOid,
                                                   toast_options,
                                                   false);
                    }

diff --git a/src/include/catalog/toasting.h b/src/include/catalog/toasting.h
index f363cb0cae62a64f6ad354a5cb6a7474c0fede24..5f4a4a79e2966a4b11b5c116b428888b5fb28ca5 100644 (file)
--- a/src/include/catalog/toasting.h
+++ b/src/include/catalog/toasting.h
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2009, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/catalog/toasting.h,v 1.7 2009/05/07 22:58:28 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/catalog/toasting.h,v 1.8 2009/06/11 20:46:11 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -17,7 +17,8 @@
 /*
  * toasting.c prototypes
  */
-extern void AlterTableCreateToastTable(Oid relOid, Datum reloptions, bool force);
+extern void AlterTableCreateToastTable(Oid relOid, Oid toastOid,
+                          Datum reloptions, bool force);
 extern void BootstrapToastTable(char *relName,
                    Oid toastOid, Oid toastIndexOid);