Reimplement temp tables using schemas. The temp table map is history;
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 31 Mar 2002 06:26:32 +0000 (06:26 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 31 Mar 2002 06:26:32 +0000 (06:26 +0000)
temp table entries in pg_class have the names the user would expect.

29 files changed:
src/backend/access/transam/xact.c
src/backend/access/transam/xlogutils.c
src/backend/bootstrap/bootparse.y
src/backend/catalog/aclchk.c
src/backend/catalog/heap.c
src/backend/catalog/index.c
src/backend/catalog/namespace.c
src/backend/catalog/pg_namespace.c
src/backend/commands/cluster.c
src/backend/commands/command.c
src/backend/commands/creatinh.c
src/backend/commands/indexcmds.c
src/backend/commands/rename.c
src/backend/commands/trigger.c
src/backend/commands/vacuum.c
src/backend/executor/execMain.c
src/backend/parser/analyze.c
src/backend/tcop/utility.c
src/backend/utils/cache/Makefile
src/backend/utils/cache/relcache.c
src/backend/utils/cache/syscache.c
src/backend/utils/cache/temprel.c [deleted file]
src/backend/utils/init/postinit.c
src/include/catalog/heap.h
src/include/catalog/index.h
src/include/catalog/namespace.h
src/include/catalog/pg_namespace.h
src/include/utils/rel.h
src/include/utils/temprel.h [deleted file]

index e18a7de2d07ec4a5e01d9c55b18f0aaa30e44730..046f3d52aad9cd85d664a21c1dc10ab6143a3c82 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/access/transam/xact.c,v 1.118 2002/03/15 19:20:29 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/access/transam/xact.c,v 1.119 2002/03/31 06:26:29 tgl Exp $
  *
  * NOTES
  *             Transaction aborts can now occur two ways:
 #include "utils/portal.h"
 #include "utils/catcache.h"
 #include "utils/relcache.h"
-#include "utils/temprel.h"
-
 #include "pgstat.h"
 
+
 extern bool SharedBufferChanged;
 
 static void AbortTransaction(void);
@@ -999,7 +998,6 @@ CommitTransaction(void)
         */
 
        RelationPurgeLocalRelation(true);
-       AtEOXact_temp_relations(true);
        smgrDoPendingDeletes(true);
 
        AtEOXact_SPI();
@@ -1102,7 +1100,6 @@ AbortTransaction(void)
        }
 
        RelationPurgeLocalRelation(false);
-       AtEOXact_temp_relations(false);
        smgrDoPendingDeletes(false);
 
        AtEOXact_SPI();
index 8a9554b2a6d563c1c101cf959caf30b42dcbbe49..ae79ba85b1d3c456553a7a6df3cee9f455cae0db 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Header: /cvsroot/pgsql/src/backend/access/transam/xlogutils.c,v 1.22 2002/03/02 21:39:20 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/access/transam/xlogutils.c,v 1.23 2002/03/31 06:26:29 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -335,7 +335,7 @@ XLogOpenRelation(bool redo, RmgrId rmid, RelFileNode rnode)
        {
                res = _xl_new_reldesc();
 
-               sprintf(RelationGetPhysicalRelationName(&(res->reldata)), "%u", rnode.relNode);
+               sprintf(RelationGetRelationName(&(res->reldata)), "%u", rnode.relNode);
 
                /* unexisting DB id */
                res->reldata.rd_lockInfo.lockRelId.dbId = RecoveryDb;
index 3b0d34f9fcd8450caaa989fb0ce8666d83341a02..33c18f3b2d2513ce9286b9201dd485bc59f89090 100644 (file)
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/bootstrap/bootparse.y,v 1.41 2002/03/26 19:15:16 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/bootstrap/bootparse.y,v 1.42 2002/03/31 06:26:29 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -184,7 +184,7 @@ Boot_CreateStmt:
                                                reldesc = heap_create(LexIDStr($4),
                                                                                          PG_CATALOG_NAMESPACE,
                                                                                          tupdesc,
-                                                                                         false, true, true);
+                                                                                         true, true);
                                                reldesc->rd_rel->relhasoids = ! ($3);
                                                elog(DEBUG3, "bootstrap relation created");
                                        }
@@ -199,7 +199,6 @@ Boot_CreateStmt:
                                                                                                          tupdesc,
                                                                                                          RELKIND_RELATION,
                                                                                                          ! ($3),
-                                                                                                         false,
                                                                                                          true);
                                                elog(DEBUG3, "relation created with oid %u", id);
                                        }
index 3269964aa079f7c1ea183328498d9413f3f727c7..00c52e89c4b80da4f5ff2836733ff1c75997d278 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/catalog/aclchk.c,v 1.60 2002/03/29 19:05:59 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/catalog/aclchk.c,v 1.61 2002/03/31 06:26:29 tgl Exp $
  *
  * NOTES
  *       See acl.h.
@@ -40,7 +40,6 @@
 #include "parser/parse_type.h"
 #include "utils/acl.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
 static void ExecuteGrantStmt_Table(GrantStmt *stmt);
@@ -811,7 +810,6 @@ pg_class_aclcheck(Oid table_oid, Oid userid, AclMode mode)
        relname = NameStr(((Form_pg_class) GETSTRUCT(tuple))->relname);
        if ((mode & (ACL_INSERT | ACL_UPDATE | ACL_DELETE)) &&
                !allowSystemTableMods && IsSystemRelationName(relname) &&
-               !is_temp_relname(relname) &&
                !usecatupd)
        {
 #ifdef ACLDEBUG
index fa64676e349f338ddd5495dfcf9059882ce4fe46..61688793cf4d14b7958065fe9fb5ae1b421850bd 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/catalog/heap.c,v 1.193 2002/03/29 19:05:59 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/catalog/heap.c,v 1.194 2002/03/31 06:26:29 tgl Exp $
  *
  *
  * INTERFACE ROUTINES
 #include "utils/lsyscache.h"
 #include "utils/relcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
 static void AddNewRelationTuple(Relation pg_class_desc,
                                        Relation new_rel_desc,
                                        Oid new_rel_oid, Oid new_type_oid,
-                                       char relkind, bool relhasoids,
-                                       char *temp_relname);
+                                       char relkind, bool relhasoids);
 static void DeleteAttributeTuples(Relation rel);
 static void DeleteRelationTuple(Relation rel);
 static void DeleteTypeTuple(Relation rel);
@@ -205,34 +203,29 @@ SystemAttributeByName(const char *attname, bool relhasoids)
  *
  *             Remove the system relation specific code to elsewhere eventually.
  *
- * NOTE: if istemp is TRUE then heap_create will overwrite relname with
- * the unique "real" name chosen for the temp relation.
- *
  * If storage_create is TRUE then heap_storage_create is called here,
  * else caller must call heap_storage_create later.
  * ----------------------------------------------------------------
  */
 Relation
-heap_create(char *relname,
+heap_create(const char *relname,
                        Oid relnamespace,
                        TupleDesc tupDesc,
-                       bool istemp,
                        bool storage_create,
                        bool allow_system_table_mods)
 {
-       static unsigned int uniqueId = 0;
-
        Oid                     relid;
        Oid                     dbid = MyDatabaseId;
-       RelFileNode     rnode;
        bool            nailme = false;
+       RelFileNode     rnode;
        Relation        rel;
 
        /*
         * sanity checks
         */
-       if (relname && !allow_system_table_mods &&
-               IsSystemRelationName(relname) && IsNormalProcessingMode())
+       if (!allow_system_table_mods &&
+               IsSystemRelationName(relname) &&
+               IsNormalProcessingMode())
                elog(ERROR, "invalid relation name \"%s\"; "
                         "the 'pg_' name prefix is reserved for system catalogs",
                         relname);
@@ -291,16 +284,6 @@ heap_create(char *relname,
        else
                relid = newoid();
 
-       if (istemp)
-       {
-               /*
-                * replace relname of caller with a unique name for a temp
-                * relation
-                */
-               snprintf(relname, NAMEDATALEN, "%s_%d_%u",
-                                PG_TEMP_REL_PREFIX, (int) MyProcPid, uniqueId++);
-       }
-
        /*
         * For now, the physical identifier of the relation is the same as the
         * logical identifier.
@@ -528,8 +511,7 @@ AddNewRelationTuple(Relation pg_class_desc,
                                        Oid new_rel_oid,
                                        Oid new_type_oid,
                                        char relkind,
-                                       bool relhasoids,
-                                       char *temp_relname)
+                                       bool relhasoids)
 {
        Form_pg_class new_rel_reltup;
        HeapTuple       tup;
@@ -599,9 +581,6 @@ AddNewRelationTuple(Relation pg_class_desc,
         */
        heap_insert(pg_class_desc, tup);
 
-       if (temp_relname)
-               create_temp_relation(temp_relname, tup);
-
        if (!IsIgnoringSystemIndexes())
        {
                /*
@@ -669,19 +648,17 @@ AddNewRelationType(const char *typeName,
  * --------------------------------
  */
 Oid
-heap_create_with_catalog(char *relname,
+heap_create_with_catalog(const char *relname,
                                                 Oid relnamespace,
                                                 TupleDesc tupdesc,
                                                 char relkind,
                                                 bool relhasoids,
-                                                bool istemp,
                                                 bool allow_system_table_mods)
 {
        Relation        pg_class_desc;
        Relation        new_rel_desc;
        Oid                     new_rel_oid;
        Oid                     new_type_oid;
-       char       *temp_relname = NULL;
 
        /*
         * sanity checks
@@ -693,32 +670,17 @@ heap_create_with_catalog(char *relname,
 
        CheckAttributeNames(tupdesc, relhasoids);
 
-       /* temp tables can mask non-temp tables */
-       if ((!istemp && get_relname_relid(relname, relnamespace)) ||
-               (istemp && is_temp_rel_name(relname)))
+       if (get_relname_relid(relname, relnamespace))
                elog(ERROR, "Relation '%s' already exists", relname);
 
-       if (istemp)
-       {
-               /* save user relation name because heap_create changes it */
-               temp_relname = pstrdup(relname);                /* save original value */
-               relname = palloc(NAMEDATALEN);
-               strcpy(relname, temp_relname);  /* heap_create will change this */
-       }
-
        /*
         * Tell heap_create not to create a physical file; we'll do that below
         * after all our catalog updates are done.      (This isn't really
         * necessary anymore, but we may as well avoid the cycles of creating
         * and deleting the file in case we fail.)
-        *
-        * Note: The call to heap_create() changes relname for temp tables; it
-        * becomes the true physical relname. The call to
-        * heap_storage_create() does all the "real" work of creating the disk
-        * file for the relation.
         */
        new_rel_desc = heap_create(relname, relnamespace, tupdesc,
-                                                          istemp, false, allow_system_table_mods);
+                                                          false, allow_system_table_mods);
 
        /* Fetch the relation OID assigned by heap_create */
        new_rel_oid = new_rel_desc->rd_att->attrs[0]->attrelid;
@@ -740,8 +702,7 @@ heap_create_with_catalog(char *relname,
                                                new_rel_oid,
                                                new_type_oid,
                                                relkind,
-                                               relhasoids,
-                                               temp_relname);
+                                               relhasoids);
 
        /*
         * since defining a relation also defines a complex type, we add a new
@@ -780,12 +741,6 @@ heap_create_with_catalog(char *relname,
        heap_close(new_rel_desc, NoLock);       /* do not unlock till end of xact */
        heap_close(pg_class_desc, RowExclusiveLock);
 
-       if (istemp)
-       {
-               pfree(relname);
-               pfree(temp_relname);
-       }
-
        return new_rel_oid;
 }
 
@@ -1226,26 +1181,19 @@ heap_drop_with_catalog(Oid rid,
 {
        Relation        rel;
        Oid                     toasttableOid;
-       bool            has_toasttable;
-       bool            istemp;
        int                     i;
 
        /*
         * Open and lock the relation.
         */
        rel = heap_open(rid, AccessExclusiveLock);
-       has_toasttable = rel->rd_rel->reltoastrelid != InvalidOid;
        toasttableOid = rel->rd_rel->reltoastrelid;
-       istemp = is_temp_rel_name(RelationGetRelationName(rel));
 
        /*
         * prevent deletion of system relations
         */
-       /* allow temp of pg_class? Guess so. */
-       if (!istemp &&
-               !allow_system_table_mods &&
-               IsSystemRelationName(RelationGetRelationName(rel)) &&
-               !is_temp_relname(RelationGetRelationName(rel)))
+       if (!allow_system_table_mods &&
+               IsSystemRelationName(RelationGetRelationName(rel)))
                elog(ERROR, "System relation \"%s\" may not be dropped",
                         RelationGetRelationName(rel));
 
@@ -1319,11 +1267,8 @@ heap_drop_with_catalog(Oid rid,
         */
        RelationForgetRelation(rid);
 
-       /* and from the temp-table map */
-       if (istemp)
-               remove_temp_rel_by_relid(rid);
-
-       if (has_toasttable)
+       /* If it has a toast table, recurse to get rid of that too */
+       if (OidIsValid(toasttableOid))
                heap_drop_with_catalog(toasttableOid, true);
 }
 
index 17d290f84ebb835e24641cec1ee9d32b18d93a3c..f8a1e5d4b20ab7c8008f5499cd135747e3ad892c 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/catalog/index.c,v 1.174 2002/03/26 19:15:28 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/catalog/index.c,v 1.175 2002/03/31 06:26:29 tgl Exp $
  *
  *
  * INTERFACE ROUTINES
@@ -33,6 +33,7 @@
 #include "catalog/index.h"
 #include "catalog/indexing.h"
 #include "catalog/pg_index.h"
+#include "catalog/pg_namespace.h"
 #include "catalog/pg_opclass.h"
 #include "catalog/pg_proc.h"
 #include "catalog/pg_type.h"
@@ -51,7 +52,6 @@
 #include "utils/lsyscache.h"
 #include "utils/relcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
 /*
@@ -69,7 +69,7 @@ static TupleDesc ConstructTupleDescriptor(Relation heapRelation,
                                                 int numatts, AttrNumber *attNums,
                                                 Oid *classObjectId);
 static void ConstructIndexReldesc(Relation indexRelation, Oid amoid);
-static void UpdateRelationRelation(Relation indexRelation, char *temp_relname);
+static void UpdateRelationRelation(Relation indexRelation);
 static void InitializeAttributeOids(Relation indexRelation,
                                                int numatts, Oid indexoid);
 static void AppendAttributeTuples(Relation indexRelation, int numatts);
@@ -320,7 +320,8 @@ ConstructIndexReldesc(Relation indexRelation, Oid amoid)
        indexRelation->rd_rel->relowner = GetUserId();
        indexRelation->rd_rel->relam = amoid;
        indexRelation->rd_rel->relisshared =
-               IsSharedSystemRelationName(RelationGetPhysicalRelationName(indexRelation));
+               (RelationGetNamespace(indexRelation) == PG_CATALOG_NAMESPACE) &&
+               IsSharedSystemRelationName(RelationGetRelationName(indexRelation));
        indexRelation->rd_rel->relkind = RELKIND_INDEX;
        indexRelation->rd_rel->relhasoids = false;
 }
@@ -330,7 +331,7 @@ ConstructIndexReldesc(Relation indexRelation, Oid amoid)
  * ----------------------------------------------------------------
  */
 static void
-UpdateRelationRelation(Relation indexRelation, char *temp_relname)
+UpdateRelationRelation(Relation indexRelation)
 {
        Relation        pg_class;
        HeapTuple       tuple;
@@ -350,16 +351,12 @@ UpdateRelationRelation(Relation indexRelation, char *temp_relname)
        tuple->t_data->t_oid = RelationGetRelid(indexRelation);
        heap_insert(pg_class, tuple);
 
-       if (temp_relname)
-               create_temp_relation(temp_relname, tuple);
-
        /*
         * During normal processing, we need to make sure that the system
         * catalog indices are correct.  Bootstrap (initdb) time doesn't
         * require this, because we make sure that the indices are correct
         * just before exiting.
         */
-
        if (!IsIgnoringSystemIndexes())
        {
                CatalogOpenIndices(Num_pg_class_indices, Name_pg_class_indices, idescs);
@@ -555,11 +552,10 @@ UpdateIndexRelation(Oid indexoid,
  */
 Oid
 index_create(Oid heapRelationId,
-                        char *indexRelationName,
+                        const char *indexRelationName,
                         IndexInfo *indexInfo,
                         Oid accessMethodObjectId,
                         Oid *classObjectId,
-                        bool istemp,
                         bool primary,
                         bool allow_system_table_mods)
 {
@@ -568,7 +564,6 @@ index_create(Oid heapRelationId,
        TupleDesc       indexTupDesc;
        Oid                     namespaceId;
        Oid                     indexoid;
-       char       *temp_relname = NULL;
 
        SetReindexProcessing(false);
 
@@ -591,8 +586,7 @@ index_create(Oid heapRelationId,
                IsNormalProcessingMode())
                elog(ERROR, "User-defined indexes on system catalogs are not supported");
 
-       if ((!istemp && get_relname_relid(indexRelationName, namespaceId)) ||
-               (istemp && is_temp_rel_name(indexRelationName)))
+       if (get_relname_relid(indexRelationName, namespaceId))
                elog(ERROR, "index named \"%s\" already exists",
                         indexRelationName);
 
@@ -608,22 +602,14 @@ index_create(Oid heapRelationId,
                                                                                        indexInfo->ii_KeyAttrNumbers,
                                                                                                classObjectId);
 
-       if (istemp)
-       {
-               /* save user relation name because heap_create changes it */
-               temp_relname = pstrdup(indexRelationName);              /* save original */
-               indexRelationName = palloc(NAMEDATALEN);
-               strcpy(indexRelationName, temp_relname);                /* heap_create will
-                                                                                                                * change this */
-       }
-
        /*
-        * create the index relation
+        * create the index relation (but don't create storage yet)
         */
        indexRelation = heap_create(indexRelationName,
                                                                namespaceId,
                                                                indexTupDesc,
-                                                               istemp, false, allow_system_table_mods);
+                                                               false,
+                                                               allow_system_table_mods);
        indexoid = RelationGetRelid(indexRelation);
 
        /*
@@ -645,7 +631,7 @@ index_create(Oid heapRelationId,
         *        (append RELATION tuple)
         * ----------------
         */
-       UpdateRelationRelation(indexRelation, temp_relname);
+       UpdateRelationRelation(indexRelation);
 
        /*
         * We create the disk file for this relation here
@@ -835,9 +821,6 @@ index_drop(Oid indexId)
        heap_close(userHeapRelation, NoLock);
 
        RelationForgetRelation(indexId);
-
-       /* if it's a temp index, clear the temp mapping table entry */
-       remove_temp_rel_by_relid(indexId);
 }
 
 /* ----------------------------------------------------------------
index e245ae5a3e43fa38df5dec0d340c12c32c75081b..a1be64bd80e96f96572aad34b7199dd2ffffb212 100644 (file)
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/catalog/namespace.c,v 1.3 2002/03/30 01:02:41 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/catalog/namespace.c,v 1.4 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
 #include "postgres.h"
 
+#include "access/heapam.h"
+#include "access/xact.h"
+#include "catalog/catname.h"
+#include "catalog/heap.h"
 #include "catalog/namespace.h"
+#include "catalog/pg_inherits.h"
 #include "catalog/pg_namespace.h"
+#include "catalog/pg_shadow.h"
 #include "miscadmin.h"
 #include "nodes/makefuncs.h"
+#include "storage/backendid.h"
+#include "utils/fmgroids.h"
 #include "utils/lsyscache.h"
 #include "utils/syscache.h"
 
 
+/*
+ * The namespace search path is a possibly-empty list of namespace OIDs.
+ * In addition to the explicit list, the TEMP table namespace is always
+ * implicitly searched first (if it's been initialized).  Also, the system
+ * catalog namespace is always searched.  If the system namespace is
+ * explicitly present in the path then it will be searched in the specified
+ * order; otherwise it will be searched after TEMP tables and *before* the
+ * explicit list.  (It might seem that the system namespace should be
+ * implicitly last, but this behavior appears to be required by SQL99.
+ * Also, this provides a way to search the system namespace first without
+ * thereby making it the default creation target namespace.)
+ *
+ * The default creation target namespace is kept equal to the first element
+ * of the explicit list, or is the system namespace if the list is empty.
+ *
+ * In bootstrap mode or a standalone backend, the default search path is
+ * empty, so that the system namespace is the only one searched or inserted
+ * into.  In multiuser mode, the default search path contains the PG_PUBLIC
+ * namespace, preceded by the user's own namespace if one exists.
+ */
+
+static List *namespaceSearchPath = NIL;
+
+/* this flag must be updated correctly when namespaceSearchPath is changed */
+static bool pathContainsSystemNamespace = false;
+
+/* default place to create stuff */
+static Oid     defaultCreationNamespace = PG_CATALOG_NAMESPACE;
+
+/*
+ * myTempNamespace is InvalidOid until and unless a TEMP namespace is set up
+ * in a particular backend session (this happens when a CREATE TEMP TABLE
+ * command is first executed).  Thereafter it's the OID of the temp namespace.
+ */
+static Oid     myTempNamespace = InvalidOid;
+
+
+/*
+ * Deletion ordering constraint item.
+ */
+typedef struct DelConstraint
+{
+       Oid                     referencer;             /* table to delete first */
+       Oid                     referencee;             /* table to delete second */
+       int                     pred;                   /* workspace for TopoSortRels */
+       struct DelConstraint *link;     /* workspace for TopoSortRels */
+} DelConstraint;
+
+
+/* Local functions */
+static Oid     GetTempTableNamespace(void);
+static void RemoveTempRelations(Oid tempNamespaceId);
+static List *FindTempRelations(Oid tempNamespaceId);
+static List *FindDeletionConstraints(List *relOids);
+static List *TopoSortRels(List *relOids, List *constraintList);
+static void RemoveTempRelationsCallback(void);
+
+
 /*
  * RangeVarGetRelid
  *             Given a RangeVar describing an existing relation,
@@ -52,6 +118,7 @@ RangeVarGetRelid(const RangeVar *relation, bool failOK)
 
        if (relation->schemaname)
        {
+               /* use exact schema given */
                namespaceId = GetSysCacheOid(NAMESPACENAME,
                                                                         CStringGetDatum(relation->schemaname),
                                                                         0, 0, 0);
@@ -62,6 +129,7 @@ RangeVarGetRelid(const RangeVar *relation, bool failOK)
        }
        else
        {
+               /* search the namespace path */
                relId = RelnameGetRelid(relation->relname);
        }
 
@@ -100,8 +168,20 @@ RangeVarGetCreationNamespace(const RangeVar *newRelation)
                        elog(ERROR, "Cross-database references are not implemented");
        }
 
+       if (newRelation->istemp)
+       {
+               /* TEMP tables are created in our backend-local temp namespace */
+               if (newRelation->schemaname)
+                       elog(ERROR, "TEMP tables may not specify a namespace");
+               /* Initialize temp namespace if first time through */
+               if (!OidIsValid(myTempNamespace))
+                       myTempNamespace = GetTempTableNamespace();
+               return myTempNamespace;
+       }
+
        if (newRelation->schemaname)
        {
+               /* use exact schema given */
                namespaceId = GetSysCacheOid(NAMESPACENAME,
                                                                         CStringGetDatum(newRelation->schemaname),
                                                                         0, 0, 0);
@@ -111,8 +191,8 @@ RangeVarGetCreationNamespace(const RangeVar *newRelation)
        }
        else
        {
-               /* XXX Wrong!  Need to get a default schema from somewhere */
-               namespaceId = PG_CATALOG_NAMESPACE;
+               /* use the default creation namespace */
+               namespaceId = defaultCreationNamespace;
        }
 
        return namespaceId;
@@ -126,23 +206,91 @@ RangeVarGetCreationNamespace(const RangeVar *newRelation)
 Oid
 RelnameGetRelid(const char *relname)
 {
-       /* XXX Wrong!  must search search path */
-       return get_relname_relid(relname, PG_CATALOG_NAMESPACE);
+       Oid                     relid;
+       List       *lptr;
+
+       /*
+        * If a TEMP-table namespace has been set up, it is implicitly first
+        * in the search path.
+        */
+       if (OidIsValid(myTempNamespace))
+       {
+               relid = get_relname_relid(relname, myTempNamespace);
+               if (OidIsValid(relid))
+                       return relid;
+       }
+
+       /*
+        * If system namespace is not in path, implicitly search it before path
+        */
+       if (!pathContainsSystemNamespace)
+       {
+               relid = get_relname_relid(relname, PG_CATALOG_NAMESPACE);
+               if (OidIsValid(relid))
+                       return relid;
+       }
+
+       /*
+        * Else search the path
+        */
+       foreach(lptr, namespaceSearchPath)
+       {
+               Oid                     namespaceId = (Oid) lfirsti(lptr);
+
+               relid = get_relname_relid(relname, namespaceId);
+               if (OidIsValid(relid))
+                       return relid;
+       }
+
+       /* Not found in path */
+       return InvalidOid;
 }
 
 /*
  * TypenameGetTypid
  *             Try to resolve an unqualified datatype name.
  *             Returns OID if type found in search path, else InvalidOid.
+ *
+ * This is essentially the same as RelnameGetRelid, but we never search
+ * the TEMP table namespace --- there is no reason to refer to the types
+ * of temp tables, AFAICS.
  */
 Oid
 TypenameGetTypid(const char *typname)
 {
-       /* XXX wrong, should use namespace search */
-       return GetSysCacheOid(TYPENAMENSP,
-                                                 PointerGetDatum(typname),
-                                                 ObjectIdGetDatum(PG_CATALOG_NAMESPACE),
-                                                 0, 0);
+       Oid                     typid;
+       List       *lptr;
+
+       /*
+        * If system namespace is not in path, implicitly search it before path
+        */
+       if (!pathContainsSystemNamespace)
+       {
+               typid = GetSysCacheOid(TYPENAMENSP,
+                                                          PointerGetDatum(typname),
+                                                          ObjectIdGetDatum(PG_CATALOG_NAMESPACE),
+                                                          0, 0);
+               if (OidIsValid(typid))
+                       return typid;
+       }
+
+       /*
+        * Else search the path
+        */
+       foreach(lptr, namespaceSearchPath)
+       {
+               Oid                     namespaceId = (Oid) lfirsti(lptr);
+
+               typid = GetSysCacheOid(TYPENAMENSP,
+                                                          PointerGetDatum(typname),
+                                                          ObjectIdGetDatum(namespaceId),
+                                                          0, 0);
+               if (OidIsValid(typid))
+                       return typid;
+       }
+
+       /* Not found in path */
+       return InvalidOid;
 }
 
 /*
@@ -150,6 +298,9 @@ TypenameGetTypid(const char *typname)
  *             Given a possibly-qualified name for an object (in List-of-Values
  *             format), determine what namespace the object should be created in.
  *             Also extract and return the object name (last component of list).
+ *
+ * This is *not* used for tables.  Hence, the TEMP table namespace is
+ * never selected as the creation target.
  */
 Oid
 QualifiedNameGetCreationNamespace(List *names, char **objname_p)
@@ -186,6 +337,7 @@ QualifiedNameGetCreationNamespace(List *names, char **objname_p)
 
        if (schemaname)
        {
+               /* use exact schema given */
                namespaceId = GetSysCacheOid(NAMESPACENAME,
                                                                         CStringGetDatum(schemaname),
                                                                         0, 0, 0);
@@ -195,8 +347,8 @@ QualifiedNameGetCreationNamespace(List *names, char **objname_p)
        }
        else
        {
-               /* XXX Wrong!  Need to get a default schema from somewhere */
-               namespaceId = PG_CATALOG_NAMESPACE;
+               /* use the default creation namespace */
+               namespaceId = defaultCreationNamespace;
        }
 
        *objname_p = objname;
@@ -233,3 +385,320 @@ makeRangeVarFromNameList(List *names)
 
        return rel;
 }
+
+/*
+ * isTempNamespace - is the given namespace my temporary-table namespace?
+ */
+bool
+isTempNamespace(Oid namespaceId)
+{
+       if (OidIsValid(myTempNamespace) && myTempNamespace == namespaceId)
+               return true;
+       return false;
+}
+
+/*
+ * GetTempTableNamespace
+ *             Initialize temp table namespace on first use in a particular backend
+ */
+static Oid
+GetTempTableNamespace(void)
+{
+       char            namespaceName[NAMEDATALEN];
+       Oid                     namespaceId;
+
+       snprintf(namespaceName, NAMEDATALEN, "pg_temp_%d", MyBackendId);
+
+       namespaceId = GetSysCacheOid(NAMESPACENAME,
+                                                                CStringGetDatum(namespaceName),
+                                                                0, 0, 0);
+       if (!OidIsValid(namespaceId))
+       {
+               /*
+                * First use of this temp namespace in this database; create it.
+                * The temp namespaces are always owned by the superuser.
+                */
+               namespaceId = NamespaceCreate(namespaceName, BOOTSTRAP_USESYSID);
+               /* Advance command counter to make namespace visible */
+               CommandCounterIncrement();
+       }
+       else
+       {
+               /*
+                * If the namespace already exists, clean it out (in case the
+                * former owner crashed without doing so).
+                */
+               RemoveTempRelations(namespaceId);
+       }
+
+       /*
+        * Register exit callback to clean out temp tables at backend shutdown.
+        */
+       on_shmem_exit(RemoveTempRelationsCallback, 0);
+
+       return namespaceId;
+}
+
+/*
+ * Remove all relations in the specified temp namespace.
+ *
+ * This is called at backend shutdown (if we made any temp relations).
+ * It is also called when we begin using a pre-existing temp namespace,
+ * in order to clean out any relations that might have been created by
+ * a crashed backend.
+ */
+static void
+RemoveTempRelations(Oid tempNamespaceId)
+{
+       List       *tempRelList;
+       List       *constraintList;
+       List       *lptr;
+
+       /* Get a list of relations to delete */
+       tempRelList = FindTempRelations(tempNamespaceId);
+
+       if (tempRelList == NIL)
+               return;                                 /* nothing to do */
+
+       /* If more than one, sort them to respect any deletion-order constraints */
+       if (length(tempRelList) > 1)
+       {
+               constraintList = FindDeletionConstraints(tempRelList);
+               if (constraintList != NIL)
+                       tempRelList = TopoSortRels(tempRelList, constraintList);
+       }
+
+       /* Scan the list and delete all entries */
+       foreach(lptr, tempRelList)
+       {
+               Oid                     reloid = (Oid) lfirsti(lptr);
+
+               heap_drop_with_catalog(reloid, true);
+               /*
+                * Advance cmd counter to make catalog changes visible, in case
+                * a later entry depends on this one.
+                */
+               CommandCounterIncrement();
+       }
+}
+
+/*
+ * Find all relations in the specified temp namespace.
+ *
+ * Returns a list of relation OIDs.
+ */
+static List *
+FindTempRelations(Oid tempNamespaceId)
+{
+       List       *tempRelList = NIL;
+       Relation        pgclass;
+       HeapScanDesc scan;
+       HeapTuple       tuple;
+       ScanKeyData key;
+
+       /*
+        * Scan pg_class to find all the relations in the target namespace.
+        * Ignore indexes, though, on the assumption that they'll go away
+        * when their tables are deleted.
+        */
+       ScanKeyEntryInitialize(&key, 0x0,
+                                                  Anum_pg_class_relnamespace,
+                                                  F_OIDEQ,
+                                                  ObjectIdGetDatum(tempNamespaceId));
+
+       pgclass = heap_openr(RelationRelationName, AccessShareLock);
+       scan = heap_beginscan(pgclass, false, SnapshotNow, 1, &key);
+
+       while (HeapTupleIsValid(tuple = heap_getnext(scan, 0)))
+       {
+               switch (((Form_pg_class) GETSTRUCT(tuple))->relkind)
+               {
+                       case RELKIND_RELATION:
+                       case RELKIND_SEQUENCE:
+                       case RELKIND_VIEW:
+                               tempRelList = lconsi(tuple->t_data->t_oid, tempRelList);
+                               break;
+                       default:
+                               break;
+               }
+       }
+
+       heap_endscan(scan);
+       heap_close(pgclass, AccessShareLock);
+
+       return tempRelList;
+}
+
+/*
+ * Find deletion-order constraints involving the given relation OIDs.
+ *
+ * Returns a list of DelConstraint objects.
+ */
+static List *
+FindDeletionConstraints(List *relOids)
+{
+       List       *constraintList = NIL;
+       Relation        inheritsrel;
+       HeapScanDesc scan;
+       HeapTuple       tuple;
+
+       /*
+        * Scan pg_inherits to find parents and children that are in the list.
+        */
+       inheritsrel = heap_openr(InheritsRelationName, AccessShareLock);
+       scan = heap_beginscan(inheritsrel, 0, SnapshotNow, 0, NULL);
+
+       while (HeapTupleIsValid(tuple = heap_getnext(scan, 0)))
+       {
+               Oid             inhrelid = ((Form_pg_inherits) GETSTRUCT(tuple))->inhrelid;
+               Oid             inhparent = ((Form_pg_inherits) GETSTRUCT(tuple))->inhparent;
+
+               if (intMember(inhrelid, relOids) && intMember(inhparent, relOids))
+               {
+                       DelConstraint  *item;
+
+                       item = (DelConstraint *) palloc(sizeof(DelConstraint));
+                       item->referencer = inhrelid;
+                       item->referencee = inhparent;
+                       constraintList = lcons(item, constraintList);
+               }
+       }
+
+       heap_endscan(scan);
+       heap_close(inheritsrel, AccessShareLock);
+
+       return constraintList;
+}
+
+/*
+ * TopoSortRels -- topological sort of a list of rels to delete
+ *
+ * This is a lot simpler and slower than, for example, the topological sort
+ * algorithm shown in Knuth's Volume 1.  However, we are not likely to be
+ * working with more than a few constraints, so the apparent slowness of the
+ * algorithm won't really matter.
+ */
+static List *
+TopoSortRels(List *relOids, List *constraintList)
+{
+       int                     queue_size = length(relOids);
+       Oid                *rels;
+       int                *beforeConstraints;
+       DelConstraint **afterConstraints;
+       List       *resultList = NIL;
+       List       *lptr;
+       int                     i,
+                               j,
+                               k,
+                               last;
+
+       /* Allocate workspace */
+       rels = (Oid *) palloc(queue_size * sizeof(Oid));
+       beforeConstraints = (int *) palloc(queue_size * sizeof(int));
+       afterConstraints = (DelConstraint **)
+               palloc(queue_size * sizeof(DelConstraint*));
+
+       /* Build an array of the target relation OIDs */
+       i = 0;
+       foreach(lptr, relOids)
+       {
+               rels[i++] = (Oid) lfirsti(lptr);
+       }
+
+       /*
+        * Scan the constraints, and for each rel in the array, generate a
+        * count of the number of constraints that say it must be before
+        * something else, plus a list of the constraints that say it must be
+        * after something else. The count for the j'th rel is stored in
+        * beforeConstraints[j], and the head of its list in
+        * afterConstraints[j].  Each constraint stores its list link in
+        * its link field (note any constraint will be in just one list).
+        * The array index for the before-rel of each constraint is
+        * remembered in the constraint's pred field.
+        */
+       MemSet(beforeConstraints, 0, queue_size * sizeof(int));
+       MemSet(afterConstraints, 0, queue_size * sizeof(DelConstraint*));
+       foreach(lptr, constraintList)
+       {
+               DelConstraint  *constraint = (DelConstraint *) lfirst(lptr);
+               Oid                     rel;
+
+               /* Find the referencer rel in the array */
+               rel = constraint->referencer;
+               for (j = queue_size; --j >= 0;)
+               {
+                       if (rels[j] == rel)
+                               break;
+               }
+               Assert(j >= 0);                 /* should have found a match */
+               /* Find the referencee rel in the array */
+               rel = constraint->referencee;
+               for (k = queue_size; --k >= 0;)
+               {
+                       if (rels[k] == rel)
+                               break;
+               }
+               Assert(k >= 0);                 /* should have found a match */
+               beforeConstraints[j]++; /* referencer must come before */
+               /* add this constraint to list of after-constraints for referencee */
+               constraint->pred = j;
+               constraint->link = afterConstraints[k];
+               afterConstraints[k] = constraint;
+       }
+       /*--------------------
+        * Now scan the rels array backwards.   At each step, output the
+        * last rel that has no remaining before-constraints, and decrease
+        * the beforeConstraints count of each of the rels it was constrained
+        * against.  (This is the right order since we are building the result
+        * list back-to-front.)
+        * i = counter for number of rels left to output
+        * j = search index for rels[]
+        * dc = temp for scanning constraint list for rel j
+        * last = last valid index in rels (avoid redundant searches)
+        *--------------------
+        */
+       last = queue_size - 1;
+       for (i = queue_size; --i >= 0;)
+       {
+               DelConstraint  *dc;
+
+               /* Find next candidate to output */
+               while (rels[last] == InvalidOid)
+                       last--;
+               for (j = last; j >= 0; j--)
+               {
+                       if (rels[j] != InvalidOid && beforeConstraints[j] == 0)
+                               break;
+               }
+               /* If no available candidate, topological sort fails */
+               if (j < 0)
+                       elog(ERROR, "TopoSortRels: failed to find a workable deletion ordering");
+               /* Output candidate, and mark it done by zeroing rels[] entry */
+               resultList = lconsi(rels[j], resultList);
+               rels[j] = InvalidOid;
+               /* Update beforeConstraints counts of its predecessors */
+               for (dc = afterConstraints[j]; dc; dc = dc->link)
+                       beforeConstraints[dc->pred]--;
+       }
+
+       /* Done */
+       return resultList;
+}
+
+/*
+ * Callback to remove temp relations at backend exit.
+ */
+static void
+RemoveTempRelationsCallback(void)
+{
+       if (OidIsValid(myTempNamespace)) /* should always be true */
+       {
+               /* Need to ensure we have a usable transaction. */
+               AbortOutOfAnyTransaction();
+               StartTransactionCommand();
+
+               RemoveTempRelations(myTempNamespace);
+
+               CommitTransactionCommand();
+       }
+}
index c340cd02f73ed6df3d30e5deabbf76aa7b8a0bf3..e2cf0128dc8a6bd819ec89483f7aaf4a5dee65cc 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/catalog/pg_namespace.c,v 1.1 2002/03/22 21:34:44 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/catalog/pg_namespace.c,v 1.2 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -18,7 +18,6 @@
 #include "catalog/catname.h"
 #include "catalog/indexing.h"
 #include "catalog/pg_namespace.h"
-#include "miscadmin.h"
 #include "utils/builtins.h"
 #include "utils/syscache.h"
 
@@ -28,7 +27,7 @@
  * ---------------
  */
 Oid
-NamespaceCreate(const char *nspName)
+NamespaceCreate(const char *nspName, int32 ownerSysId)
 {
        Relation        nspdesc;
        HeapTuple       tup;
@@ -57,7 +56,7 @@ NamespaceCreate(const char *nspName)
        }
        namestrcpy(&nname, nspName);
        values[Anum_pg_namespace_nspname - 1] = NameGetDatum(&nname);
-       values[Anum_pg_namespace_nspowner - 1] = Int32GetDatum(GetUserId());
+       values[Anum_pg_namespace_nspowner - 1] = Int32GetDatum(ownerSysId);
        nulls[Anum_pg_namespace_nspacl - 1] = 'n';
 
        nspdesc = heap_openr(NamespaceRelationName, RowExclusiveLock);
index 0d0cbbe78812035f78be240d75d0c7e8f91599ad..c801c53f406b40f5172f3275817379b009a15328 100644 (file)
@@ -15,7 +15,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/cluster.c,v 1.75 2002/03/29 22:10:33 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/cluster.c,v 1.76 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
 #include "utils/builtins.h"
 #include "utils/lsyscache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
-static Oid     copy_heap(Oid OIDOldHeap, char *NewName, bool istemp);
-static void copy_index(Oid OIDOldIndex, Oid OIDNewHeap, char *NewIndexName,
-                                          bool istemp);
+static Oid     copy_heap(Oid OIDOldHeap, char *NewName);
+static void copy_index(Oid OIDOldIndex, Oid OIDNewHeap, char *NewIndexName);
 static void rebuildheap(Oid OIDNewHeap, Oid OIDOldHeap, Oid OIDOldIndex);
 
 /*
@@ -63,7 +61,6 @@ cluster(RangeVar *oldrelation, char *oldindexname)
                                OIDNewHeap;
        Relation        OldHeap,
                                OldIndex;
-       bool            istemp;
        char            NewHeapName[NAMEDATALEN];
        char            NewIndexName[NAMEDATALEN];
        RangeVar   *NewHeap;
@@ -76,8 +73,6 @@ cluster(RangeVar *oldrelation, char *oldindexname)
        OldHeap = heap_openrv(oldrelation, AccessExclusiveLock);
        OIDOldHeap = RelationGetRelid(OldHeap);
 
-       istemp = is_temp_rel_name(oldrelation->relname);
-
        /*
         * The index is expected to be in the same namespace as the relation.
         */
@@ -105,7 +100,7 @@ cluster(RangeVar *oldrelation, char *oldindexname)
         */
        snprintf(NewHeapName, NAMEDATALEN, "temp_%u", OIDOldHeap);
 
-       OIDNewHeap = copy_heap(OIDOldHeap, NewHeapName, istemp);
+       OIDNewHeap = copy_heap(OIDOldHeap, NewHeapName);
 
        /* We do not need CommandCounterIncrement() because copy_heap did it. */
 
@@ -120,7 +115,7 @@ cluster(RangeVar *oldrelation, char *oldindexname)
        /* Create new index over the tuples of the new heap. */
        snprintf(NewIndexName, NAMEDATALEN, "temp_%u", OIDOldIndex);
 
-       copy_index(OIDOldIndex, OIDNewHeap, NewIndexName, istemp);
+       copy_index(OIDOldIndex, OIDNewHeap, NewIndexName);
 
        CommandCounterIncrement();
 
@@ -145,7 +140,7 @@ cluster(RangeVar *oldrelation, char *oldindexname)
 }
 
 static Oid
-copy_heap(Oid OIDOldHeap, char *NewName, bool istemp)
+copy_heap(Oid OIDOldHeap, char *NewName)
 {
        TupleDesc       OldHeapDesc,
                                tupdesc;
@@ -166,7 +161,6 @@ copy_heap(Oid OIDOldHeap, char *NewName, bool istemp)
                                                                                  tupdesc,
                                                                                  OldHeap->rd_rel->relkind,
                                                                                  OldHeap->rd_rel->relhasoids,
-                                                                                 istemp,
                                                                                  allowSystemTableMods);
 
        /*
@@ -188,7 +182,7 @@ copy_heap(Oid OIDOldHeap, char *NewName, bool istemp)
 }
 
 static void
-copy_index(Oid OIDOldIndex, Oid OIDNewHeap, char *NewIndexName, bool istemp)
+copy_index(Oid OIDOldIndex, Oid OIDNewHeap, char *NewIndexName)
 {
        Relation        OldIndex,
                                NewHeap;
@@ -209,7 +203,6 @@ copy_index(Oid OIDOldIndex, Oid OIDNewHeap, char *NewIndexName, bool istemp)
                                 indexInfo,
                                 OldIndex->rd_rel->relam,
                                 OldIndex->rd_index->indclass,
-                                istemp,
                                 OldIndex->rd_index->indisprimary,
                                 allowSystemTableMods);
 
index 2c029fa2ae9f28e1faf2a9cda2acdf65a224b625..b79e65960f7b37fe8b80c7b1515c611e2fee7e7b 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/Attic/command.c,v 1.168 2002/03/30 01:02:41 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/Attic/command.c,v 1.169 2002/03/31 06:26:30 tgl Exp $
  *
  * NOTES
  *       The PerformAddAttribute() code, like most of the relation
@@ -55,7 +55,7 @@
 #include "utils/lsyscache.h"
 #include "utils/syscache.h"
 #include "utils/relcache.h"
-#include "utils/temprel.h"
+
 
 static void drop_default(Oid relid, int16 attnum);
 static bool needs_toast_table(Relation rel);
@@ -1344,20 +1344,27 @@ AlterTableAddConstraint(Oid myrelid,
                                        List       *list;
                                        int                     count;
 
-                                       if (is_temp_rel_name(fkconstraint->pktable->relname) &&
-                                               !is_temp_rel_name(RelationGetRelationName(rel)))
-                                               elog(ERROR, "ALTER TABLE / ADD CONSTRAINT: Unable to reference temporary table from permanent table constraint.");
-
                                        /*
                                         * Grab an exclusive lock on the pk table, so that
                                         * someone doesn't delete rows out from under us.
+                                        *
+                                        * XXX wouldn't a lesser lock be sufficient?
                                         */
-
                                        pkrel = heap_openrv(fkconstraint->pktable,
                                                                                AccessExclusiveLock);
+
+                                       /*
+                                        * Validity checks
+                                        */
                                        if (pkrel->rd_rel->relkind != RELKIND_RELATION)
                                                elog(ERROR, "referenced table \"%s\" not a relation",
                                                         fkconstraint->pktable->relname);
+
+                                       if (isTempNamespace(RelationGetNamespace(pkrel)) &&
+                                               !isTempNamespace(RelationGetNamespace(rel)))
+                                               elog(ERROR, "ALTER TABLE / ADD CONSTRAINT: Unable to reference temporary table from permanent table constraint.");
+
+                                       /* Don't need pkrel open anymore, but hold lock */
                                        heap_close(pkrel, NoLock);
 
                                        /*
@@ -1763,8 +1770,9 @@ AlterTableCreateToastTable(Oid relOid, bool silent)
        toast_relid = heap_create_with_catalog(toast_relname,
                                                                                   PG_TOAST_NAMESPACE,
                                                                                   tupdesc,
-                                                                                  RELKIND_TOASTVALUE, false,
-                                                                                  false, true);
+                                                                                  RELKIND_TOASTVALUE,
+                                                                                  false,
+                                                                                  true);
 
        /* make the toast relation visible, else index creation will fail */
        CommandCounterIncrement();
@@ -1794,7 +1802,7 @@ AlterTableCreateToastTable(Oid relOid, bool silent)
 
        toast_idxid = index_create(toast_relid, toast_idxname, indexInfo,
                                                           BTREE_AM_OID, classObjectId,
-                                                          false, true, true);
+                                                          true, true);
 
        /*
         * Update toast rel's pg_class entry to show that it has an index. The
@@ -1971,7 +1979,7 @@ CreateSchemaCommand(CreateSchemaStmt *stmt)
        }
 
        /* Create the schema's namespace */
-       NamespaceCreate(schemaName);
+       NamespaceCreate(schemaName, owner_userid);
 
        /* Let commands in the schema-element-list know about the schema */
        CommandCounterIncrement();
index f85814cc0f4cae32a858ead32cf2520847a0c953..4a76e0c11ec58d6554b6e349a833ee25f4a62157 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/Attic/creatinh.c,v 1.94 2002/03/29 22:10:33 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/Attic/creatinh.c,v 1.95 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -30,7 +30,7 @@
 #include "parser/parse_type.h"
 #include "utils/acl.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
+
 
 /* ----------------
  *             local stuff
@@ -157,11 +157,11 @@ DefineRelation(CreateStmt *stmt, char relkind)
                }
        }
 
-       relationId = heap_create_with_catalog(relname, namespaceId,
+       relationId = heap_create_with_catalog(relname,
+                                                                                 namespaceId,
                                                                                  descriptor,
                                                                                  relkind,
                                                                                  stmt->hasoids || parentHasOids,
-                                                                                 stmt->relation->istemp,
                                                                                  allowSystemTableMods);
 
        StoreCatalogInheritance(relationId, inheritOids);
@@ -447,7 +447,7 @@ MergeAttributes(List *schema, List *supers, bool istemp,
                        elog(ERROR, "CREATE TABLE: inherited relation \"%s\" is not a table",
                                 parent->relname);
                /* Permanent rels cannot inherit from temporary ones */
-               if (!istemp && is_temp_rel_name(parent->relname))
+               if (!istemp && isTempNamespace(RelationGetNamespace(relation)))
                        elog(ERROR, "CREATE TABLE: cannot inherit from temp relation \"%s\"",
                                 parent->relname);
 
index ae3476efb24f4b4da649932c2ab5ed748a89f5cf..c931ab19278d751a7e57a233a92cff5c9b7f6b68 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/indexcmds.c,v 1.65 2002/03/26 19:15:41 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/indexcmds.c,v 1.66 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -33,7 +33,6 @@
 #include "utils/fmgroids.h"
 #include "utils/lsyscache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
 #define IsFuncIndex(ATTR_LIST) (((IndexElem*)lfirst(ATTR_LIST))->args != NIL)
@@ -74,7 +73,6 @@ DefineIndex(RangeVar *heapRelation,
        Oid                *classObjectId;
        Oid                     accessMethodId;
        Oid                     relationId;
-       bool            istemp = is_temp_rel_name(heapRelation->relname);
        Relation        rel;
        HeapTuple       tuple;
        Form_pg_am      accessMethodForm;
@@ -191,7 +189,7 @@ DefineIndex(RangeVar *heapRelation,
 
        index_create(relationId, indexRelationName,
                                 indexInfo, accessMethodId, classObjectId,
-                                istemp, primary, allowSystemTableMods);
+                                primary, allowSystemTableMods);
 
        /*
         * We update the relation's pg_class tuple even if it already has
index 0f4c83fbebf50a353ba1e33e32668b997e6950d8..7e08dade24d223be769fd188a8db12083caa15b2 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/Attic/rename.c,v 1.66 2002/03/29 19:06:07 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/Attic/rename.c,v 1.67 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -39,7 +39,6 @@
 #include "utils/lsyscache.h"
 #include "utils/relcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
 #define RI_TRIGGER_PK  1               /* is a trigger on the PK relation */
@@ -270,13 +269,6 @@ renamerel(const RangeVar *relation, const char *newrelname)
                elog(ERROR, "renamerel: Illegal class name: \"%s\" -- pg_ is reserved for system catalogs",
                         newrelname);
 
-       /*
-        * Check for renaming a temp table, which only requires altering the
-        * temp-table mapping, not the underlying table.
-        */
-       if (rename_temp_relation(relation->relname, newrelname))
-               return;                                 /* all done... */
-
        /*
         * Grab an exclusive lock on the target table or index, which we will
         * NOT release until end of transaction.
index 36229b158e216e453fba95a67a7fb4f1508e5155..c20c2637178aaad9ab78b581baa907a6c6ebfd6b 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/trigger.c,v 1.109 2002/03/29 22:10:33 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/trigger.c,v 1.110 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -471,7 +471,7 @@ RelationRemoveTriggers(Relation rel)
                Form_pg_trigger pg_trigger = (Form_pg_trigger) GETSTRUCT(tup);
 
                elog(NOTICE, "DROP TABLE implicitly drops referential integrity trigger from table \"%s\"",
-                        get_temp_rel_by_physicalname(get_rel_name(pg_trigger->tgrelid)));
+                        get_rel_name(pg_trigger->tgrelid));
 
                DropTrigger(pg_trigger->tgrelid, NameStr(pg_trigger->tgname));
 
index 41405f3654a9220c0d27f3a88d8843fdae2d9104..6553529cdcf810e5b082c78a8b8f570ba9c0b0ca 100644 (file)
@@ -13,7 +13,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/commands/vacuum.c,v 1.219 2002/03/21 23:27:22 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/commands/vacuum.c,v 1.220 2002/03/31 06:26:30 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -42,8 +42,6 @@
 #include "utils/inval.h"
 #include "utils/relcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
-
 #include "pgstat.h"
 
 
@@ -351,16 +349,9 @@ getrels(Name VacRelP, const char *stmttype)
                 * we could use the cache here, but it is clearer to use scankeys
                 * for both vacuum cases, bjm 2000/01/19
                 */
-               char       *nontemp_relname;
-
-               /* We must re-map temp table names bjm 2000-04-06 */
-               nontemp_relname = get_temp_rel_by_username(NameStr(*VacRelP));
-               if (nontemp_relname == NULL)
-                       nontemp_relname = NameStr(*VacRelP);
-
                ScanKeyEntryInitialize(&key, 0x0, Anum_pg_class_relname,
                                                           F_NAMEEQ,
-                                                          PointerGetDatum(nontemp_relname));
+                                                          PointerGetDatum(NameStr(*VacRelP)));
        }
        else
        {
index ab41dd50c3cd3de1aadc610147077adad6c02a10..5bbcf6b8ef92b5ef461462b50164dfc0092f43e7 100644 (file)
@@ -27,7 +27,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.155 2002/03/26 19:15:54 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.156 2002/03/31 06:26:31 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -724,8 +724,8 @@ InitPlan(CmdType operation, Query *parseTree, Plan *plan, EState *estate)
                                        heap_create_with_catalog(intoName,
                                                                                         namespaceId,
                                                                                         tupdesc,
-                                                                                        RELKIND_RELATION, true,
-                                                                                        parseTree->into->istemp,
+                                                                                        RELKIND_RELATION,
+                                                                                        true,
                                                                                         allowSystemTableMods);
 
                                FreeTupleDesc(tupdesc);
index d02b0bf58ed3ed1835bd7c6cd842756bc2ab7992..42e18d8ac1e56549163d7ded7d9941425cc6b676 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- *     $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.224 2002/03/29 19:06:10 tgl Exp $
+ *     $Header: /cvsroot/pgsql/src/backend/parser/analyze.c,v 1.225 2002/03/31 06:26:31 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -37,8 +37,6 @@
 #include "utils/fmgroids.h"
 #include "utils/relcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
-
 #ifdef MULTIBYTE
 #include "mb/pg_wchar.h"
 #endif
@@ -2553,7 +2551,6 @@ transformAlterTableStmt(ParseState *pstate, AlterTableStmt *stmt,
                        cxt.stmtType = "ALTER TABLE";
                        cxt.relation = stmt->relation;
                        cxt.inhRelations = NIL;
-                       cxt.relation->istemp = is_temp_rel_name(stmt->relation->relname);
                        cxt.relOid = RangeVarGetRelid(stmt->relation, false);
                        cxt.hasoids = SearchSysCacheExists(ATTNUM,
                                                                                        ObjectIdGetDatum(cxt.relOid),
@@ -2583,7 +2580,6 @@ transformAlterTableStmt(ParseState *pstate, AlterTableStmt *stmt,
                        cxt.stmtType = "ALTER TABLE";
                        cxt.relation = stmt->relation;
                        cxt.inhRelations = NIL;
-                       cxt.relation->istemp = is_temp_rel_name(stmt->relation->relname);
                        cxt.relOid = RangeVarGetRelid(stmt->relation, false);
                        cxt.hasoids = SearchSysCacheExists(ATTNUM,
                                                                                        ObjectIdGetDatum(cxt.relOid),
index 40795016ab9fb634d6cc201c2c64f2d124a25280..49d3c129f3dc339036d325d6269cdc270e605b40 100644 (file)
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/tcop/utility.c,v 1.142 2002/03/29 22:10:33 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/tcop/utility.c,v 1.143 2002/03/31 06:26:31 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -49,7 +49,6 @@
 #include "utils/acl.h"
 #include "utils/lsyscache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 #include "access/xlog.h"
 
 /*
@@ -128,8 +127,7 @@ CheckDropPermissions(RangeVar *rel, char rightkind)
                elog(ERROR, "you do not own %s \"%s\"",
                         rentry->name, rel->relname);
 
-       if (!allowSystemTableMods && IsSystemRelationName(rel->relname) &&
-               !is_temp_relname(rel->relname))
+       if (!allowSystemTableMods && IsSystemRelationName(rel->relname))
                elog(ERROR, "%s \"%s\" is a system %s",
                         rentry->name, rel->relname, rentry->name);
 
index 5ac5a06827cb1fc01af67d5737f9b29e368f23ee..eb9d3e89c0ccb3dc9cbcd4a3bc9abc96f216b012 100644 (file)
@@ -4,7 +4,7 @@
 #    Makefile for utils/cache
 #
 # IDENTIFICATION
-#    $Header: /cvsroot/pgsql/src/backend/utils/cache/Makefile,v 1.15 2001/10/06 23:21:44 tgl Exp $
+#    $Header: /cvsroot/pgsql/src/backend/utils/cache/Makefile,v 1.16 2002/03/31 06:26:31 tgl Exp $
 #
 #-------------------------------------------------------------------------
 
@@ -13,7 +13,7 @@ top_builddir = ../../../..
 include $(top_builddir)/src/Makefile.global
 
 OBJS = catcache.o inval.o relcache.o syscache.o lsyscache.o \
-       fcache.o temprel.o
+       fcache.o
 
 all: SUBSYS.o
 
index 4894cd27795813f479b5f6506059b26d500dab47..fcdffa9132e0c090db97fc420182081c0ecee45e 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.158 2002/03/26 19:16:10 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.159 2002/03/31 06:26:31 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -60,7 +60,6 @@
 #include "utils/lsyscache.h"
 #include "utils/relcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
 /*
@@ -186,7 +185,7 @@ do { \
        nodentry->reldesc = RELATION; \
        if (RelationGetNamespace(RELATION) == PG_CATALOG_NAMESPACE) \
        { \
-               char *relname = RelationGetPhysicalRelationName(RELATION); \
+               char *relname = RelationGetRelationName(RELATION); \
                RelNameCacheEnt *namehentry; \
                namehentry = (RelNameCacheEnt*)hash_search(RelationSysNameCache, \
                                                                                                   relname, \
@@ -247,7 +246,7 @@ do { \
                elog(WARNING, "trying to delete a reldesc that does not exist."); \
        if (RelationGetNamespace(RELATION) == PG_CATALOG_NAMESPACE) \
        { \
-               char *relname = RelationGetPhysicalRelationName(RELATION); \
+               char *relname = RelationGetRelationName(RELATION); \
                RelNameCacheEnt *namehentry; \
                namehentry = (RelNameCacheEnt*)hash_search(RelationSysNameCache, \
                                                                                                   relname, \
@@ -1571,18 +1570,9 @@ RelationIdGetRelation(Oid relationId)
 Relation
 RelationSysNameGetRelation(const char *relationName)
 {
-       char       *temprelname;
        Relation        rd;
        RelationBuildDescInfo buildinfo;
 
-       /*
-        * if caller is looking for a temp relation, substitute its real name;
-        * we only index temp rels by their real names.
-        */
-       temprelname = get_temp_rel_by_username(relationName);
-       if (temprelname != NULL)
-               relationName = temprelname;
-
        /*
         * first try and get a reldesc from the cache
         */
index a160e4fb1bdda03ae58a03f904b0c9fada1ed2eb..7f988782c7555f142160a205bd20b21b6aaa7e0f 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/cache/syscache.c,v 1.71 2002/03/29 19:06:15 tgl Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/cache/syscache.c,v 1.72 2002/03/31 06:26:32 tgl Exp $
  *
  * NOTES
  *       These routines allow the parser/planner/executor to perform
@@ -42,7 +42,6 @@
 #include "catalog/pg_type.h"
 #include "utils/catcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 #include "miscadmin.h"
 
 
@@ -500,29 +499,9 @@ SearchSysCache(int cacheId,
                           Datum key3,
                           Datum key4)
 {
-       if (cacheId < 0 || cacheId >= SysCacheSize)
-       {
+       if (cacheId < 0 || cacheId >= SysCacheSize ||
+               ! PointerIsValid(SysCache[cacheId]))
                elog(ERROR, "SearchSysCache: Bad cache id %d", cacheId);
-               return (HeapTuple) NULL;
-       }
-
-       Assert(PointerIsValid(SysCache[cacheId]));
-
-       /*
-        * If someone tries to look up a relname, translate temp relation
-        * names to real names.  Less obviously, apply the same translation to
-        * type names, so that the type tuple of a temp table will be found
-        * when sought.  This is a kluge ... temp table substitution should be
-        * happening at a higher level ...
-        */
-       if (cacheId == RELNAMENSP || cacheId == TYPENAMENSP)
-       {
-               char       *nontemp_relname;
-
-               nontemp_relname = get_temp_rel_by_username(DatumGetCString(key1));
-               if (nontemp_relname != NULL)
-                       key1 = CStringGetDatum(nontemp_relname);
-       }
 
        return SearchCatCache(SysCache[cacheId], key1, key2, key3, key4);
 }
diff --git a/src/backend/utils/cache/temprel.c b/src/backend/utils/cache/temprel.c
deleted file mode 100644 (file)
index 7ab609e..0000000
+++ /dev/null
@@ -1,335 +0,0 @@
-/*-------------------------------------------------------------------------
- *
- * temprel.c
- *       POSTGRES temporary relation handling
- *
- * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
- * Portions Copyright (c) 1994, Regents of the University of California
- *
- *
- * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/cache/Attic/temprel.c,v 1.36 2002/03/29 19:06:16 tgl Exp $
- *
- *-------------------------------------------------------------------------
- */
-
-/*
- * This implements temp tables by modifying the relname cache lookups
- * of pg_class.
- *
- * When a temp table is created, normal entries are made for it in pg_class,
- * pg_type, etc using a unique "physical" relation name.  We also make an
- * entry in the temp table list maintained by this module.     Subsequently,
- * relname lookups are filtered through the temp table list, and attempts
- * to look up a temp table name are changed to look up the physical name.
- * This allows temp table names to mask a regular table of the same name
- * for the duration of the session.  The temp table list is also used
- * to drop the underlying physical relations at session shutdown.
- */
-
-#include "postgres.h"
-
-#include <sys/types.h>
-
-#include "catalog/heap.h"
-#include "catalog/index.h"
-#include "miscadmin.h"
-#include "utils/temprel.h"
-
-
-/* ----------------
- *             global variables
- * ----------------
- */
-
-static List *temp_rels = NIL;
-
-typedef struct TempTable
-{
-       NameData        user_relname;   /* logical name of temp table */
-       NameData        relname;                /* underlying unique name */
-       Oid                     relid;                  /* needed properties of rel */
-       char            relkind;
-
-       /*
-        * If this entry was created during this xact, it should be deleted at
-        * xact abort.  Conversely, if this entry was deleted during this
-        * xact, it should be removed at xact commit.  We leave deleted
-        * entries in the list until commit so that we can roll back if needed
-        * --- but we ignore them for purposes of lookup!
-        */
-       bool            created_in_cur_xact;
-       bool            deleted_in_cur_xact;
-} TempTable;
-
-
-/*
- * Create a temp-relation list entry given the logical temp table name
- * and the already-created pg_class tuple for the underlying relation.
- *
- * NB: we assume a check has already been made for a duplicate logical name.
- */
-void
-create_temp_relation(const char *relname, HeapTuple pg_class_tuple)
-{
-       Form_pg_class pg_class_form = (Form_pg_class) GETSTRUCT(pg_class_tuple);
-       MemoryContext oldcxt;
-       TempTable  *temp_rel;
-
-       oldcxt = MemoryContextSwitchTo(CacheMemoryContext);
-
-       temp_rel = (TempTable *) palloc(sizeof(TempTable));
-
-       StrNCpy(NameStr(temp_rel->user_relname), relname,
-                       NAMEDATALEN);
-       StrNCpy(NameStr(temp_rel->relname), NameStr(pg_class_form->relname),
-                       NAMEDATALEN);
-       temp_rel->relid = pg_class_tuple->t_data->t_oid;
-       temp_rel->relkind = pg_class_form->relkind;
-       temp_rel->created_in_cur_xact = true;
-       temp_rel->deleted_in_cur_xact = false;
-
-       temp_rels = lcons(temp_rel, temp_rels);
-
-       MemoryContextSwitchTo(oldcxt);
-}
-
-/*
- * Remove a temp relation map entry (part of DROP TABLE on a temp table).
- * We don't actually remove the entry, just mark it dead.
- *
- * We don't have the relname for indexes, so we just pass the oid.
- */
-void
-remove_temp_rel_by_relid(Oid relid)
-{
-       List       *l;
-
-       foreach(l, temp_rels)
-       {
-               TempTable  *temp_rel = (TempTable *) lfirst(l);
-
-               if (temp_rel->relid == relid)
-                       temp_rel->deleted_in_cur_xact = true;
-
-               /*
-                * Keep scanning 'cause there could be multiple matches; see
-                * RENAME
-                */
-       }
-}
-
-/*
- * To implement ALTER TABLE RENAME on a temp table, we shouldn't touch
- * the underlying physical table at all, just change the map entry!
- *
- * This routine is invoked early in ALTER TABLE RENAME to check for
- * the temp-table case.  If oldname matches a temp table name, change
- * the mapping to the new logical name and return TRUE (or elog if
- * there is a conflict with another temp table name).  If there is
- * no match, return FALSE indicating that normal rename should proceed.
- *
- * We also reject an attempt to rename a normal table to a name in use
- * as a temp table name.  That would fail later on anyway when rename.c
- * looks for a rename conflict, but we can give a more specific error
- * message for the problem here.
- *
- * It might seem that we need to check for attempts to rename the physical
- * file underlying a temp table, but that'll be rejected anyway because
- * pg_tempXXX looks like a system table name.
- */
-bool
-rename_temp_relation(const char *oldname,
-                                        const char *newname)
-{
-       List       *l;
-
-       foreach(l, temp_rels)
-       {
-               TempTable  *temp_rel = (TempTable *) lfirst(l);
-               MemoryContext oldcxt;
-               TempTable  *new_temp_rel;
-
-               if (temp_rel->deleted_in_cur_xact)
-                       continue;                       /* ignore it if logically deleted */
-
-               if (strcmp(NameStr(temp_rel->user_relname), oldname) != 0)
-                       continue;                       /* ignore non-matching entries */
-
-               /* We are renaming a temp table --- is it OK to do so? */
-               if (is_temp_rel_name(newname))
-                       elog(ERROR, "Cannot rename temp table \"%s\": temp table \"%s\" already exists",
-                                oldname, newname);
-
-               /*
-                * Create a new mapping entry and mark the old one deleted in this
-                * xact.  One of these entries will be deleted at xact end.
-                *
-                * NOTE: the new mapping entry is inserted into the list just after
-                * the old one.  We could alternatively insert it before the old
-                * one, but that'd take more code.  It does need to be in one spot
-                * or the other, to ensure that deletion of temp rels happens in
-                * the right order during remove_all_temp_relations().
-                */
-               oldcxt = MemoryContextSwitchTo(CacheMemoryContext);
-
-               new_temp_rel = (TempTable *) palloc(sizeof(TempTable));
-               memcpy(new_temp_rel, temp_rel, sizeof(TempTable));
-
-               StrNCpy(NameStr(new_temp_rel->user_relname), newname, NAMEDATALEN);
-               new_temp_rel->created_in_cur_xact = true;
-
-               lnext(l) = lcons(new_temp_rel, lnext(l));
-
-               temp_rel->deleted_in_cur_xact = true;
-
-               MemoryContextSwitchTo(oldcxt);
-
-               return true;
-       }
-
-       /* Old name does not match any temp table name, what about new? */
-       if (is_temp_rel_name(newname))
-               elog(ERROR, "Cannot rename \"%s\" to \"%s\": a temp table by that name already exists",
-                        oldname, newname);
-
-       return false;
-}
-
-
-/*
- * Remove underlying relations for all temp rels at backend shutdown.
- */
-void
-remove_all_temp_relations(void)
-{
-       List       *l;
-
-       /* skip xact start overhead if nothing to do */
-       if (temp_rels == NIL)
-               return;
-
-       AbortOutOfAnyTransaction();
-       StartTransactionCommand();
-
-       /*
-        * Scan the list and delete all entries not already deleted. We need
-        * not worry about list entries getting deleted from under us, because
-        * remove_temp_rel_by_relid() doesn't remove entries, only mark them
-        * dead.  Note that entries will be deleted in reverse order of
-        * creation --- that's critical for cases involving inheritance.
-        */
-       foreach(l, temp_rels)
-       {
-               TempTable  *temp_rel = (TempTable *) lfirst(l);
-
-               if (temp_rel->deleted_in_cur_xact)
-                       continue;                       /* ignore it if deleted already */
-
-               if (temp_rel->relkind != RELKIND_INDEX)
-                       heap_drop_with_catalog(temp_rel->relid, allowSystemTableMods);
-               else
-                       index_drop(temp_rel->relid);
-               /* advance cmd counter to make catalog changes visible */
-               CommandCounterIncrement();
-       }
-
-       CommitTransactionCommand();
-}
-
-/*
- * Clean up temprel mapping entries during transaction commit or abort.
- *
- * During commit, remove entries that were deleted during this transaction;
- * during abort, remove those created during this transaction.
- *
- * We do not need to worry about removing the underlying physical relation;
- * that's someone else's job.
- */
-void
-AtEOXact_temp_relations(bool isCommit)
-{
-       List       *l,
-                          *prev;
-
-       prev = NIL;
-       l = temp_rels;
-       while (l != NIL)
-       {
-               TempTable  *temp_rel = (TempTable *) lfirst(l);
-
-               if (isCommit ? temp_rel->deleted_in_cur_xact :
-                       temp_rel->created_in_cur_xact)
-               {
-                       /* This entry must be removed */
-                       if (prev != NIL)
-                       {
-                               lnext(prev) = lnext(l);
-                               pfree(l);
-                               l = lnext(prev);
-                       }
-                       else
-                       {
-                               temp_rels = lnext(l);
-                               pfree(l);
-                               l = temp_rels;
-                       }
-                       pfree(temp_rel);
-               }
-               else
-               {
-                       /* This entry must be preserved */
-                       temp_rel->created_in_cur_xact = false;
-                       temp_rel->deleted_in_cur_xact = false;
-                       prev = l;
-                       l = lnext(l);
-               }
-       }
-}
-
-
-/*
- * Map user name to physical name --- returns NULL if no entry.
- *
- * This also supports testing whether a name is a temp table name;
- * see is_temp_rel_name() macro.
- */
-char *
-get_temp_rel_by_username(const char *user_relname)
-{
-       List       *l;
-
-       foreach(l, temp_rels)
-       {
-               TempTable  *temp_rel = (TempTable *) lfirst(l);
-
-               if (temp_rel->deleted_in_cur_xact)
-                       continue;                       /* ignore it if logically deleted */
-
-               if (strcmp(NameStr(temp_rel->user_relname), user_relname) == 0)
-                       return NameStr(temp_rel->relname);
-       }
-       return NULL;
-}
-
-/*
- * Map physical name to user name --- returns pstrdup'd input if no match.
- */
-char *
-get_temp_rel_by_physicalname(const char *relname)
-{
-       List       *l;
-
-       foreach(l, temp_rels)
-       {
-               TempTable  *temp_rel = (TempTable *) lfirst(l);
-
-               if (temp_rel->deleted_in_cur_xact)
-                       continue;                       /* ignore it if logically deleted */
-
-               if (strcmp(NameStr(temp_rel->relname), relname) == 0)
-                       return NameStr(temp_rel->user_relname);
-       }
-       /* needed for bootstrapping temp tables */
-       return pstrdup(relname);
-}
index 3ffb8af054142d3d25fef2c697f7695b5eb20e8c..9cd5b194d922535007e1e8319db2921a1b9fe3f0 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/init/postinit.c,v 1.100 2002/03/06 06:10:25 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/init/postinit.c,v 1.101 2002/03/31 06:26:32 tgl Exp $
  *
  *
  *-------------------------------------------------------------------------
@@ -39,7 +39,6 @@
 #include "utils/portal.h"
 #include "utils/relcache.h"
 #include "utils/syscache.h"
-#include "utils/temprel.h"
 
 
 static void ReverifyMyDatabase(const char *name);
@@ -386,20 +385,13 @@ InitPostgres(const char *dbname, const char *username)
        RelationCacheInitializePhase3();
 
        /*
-        * Set up process-exit callbacks to remove temp relations and then do
-        * pre-shutdown cleanup.  This should be last because we want
-        * shmem_exit to call these routines before the exit callbacks that
-        * are registered by buffer manager, lock manager, etc.  We need to
-        * run this code before we close down database access!
+        * Set up process-exit callback to do pre-shutdown cleanup.  This should
+        * be last because we want shmem_exit to call this routine before the exit
+        * callbacks that are registered by buffer manager, lock manager, etc.
+        * We need to run this code before we close down database access!
         */
        on_shmem_exit(ShutdownPostgres, 0);
 
-       /*
-        * because callbacks are called in reverse order, this gets done
-        * first:
-        */
-       on_shmem_exit(remove_all_temp_relations, 0);
-
        /* close the transaction we started above */
        if (!bootstrap)
                CommitTransactionCommand();
index b7313c642536dac6579c2ef9ae0cc7f4584c4623..a0ab4c708a7c75c0eebfa48130991c2c126328c8 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: heap.h,v 1.48 2002/03/29 19:06:17 tgl Exp $
+ * $Id: heap.h,v 1.49 2002/03/31 06:26:32 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -27,16 +27,19 @@ typedef struct RawColumnDefault
                                                                 * tree) */
 } RawColumnDefault;
 
-extern Relation heap_create(char *relname, Oid relnamespace,
+extern Relation heap_create(const char *relname,
+                       Oid relnamespace,
                        TupleDesc tupDesc,
-                       bool istemp, bool storage_create,
+                       bool storage_create,
                        bool allow_system_table_mods);
 
 extern void heap_storage_create(Relation rel);
 
-extern Oid heap_create_with_catalog(char *relname, Oid relnamespace,
+extern Oid heap_create_with_catalog(const char *relname,
+                                                Oid relnamespace,
                                                 TupleDesc tupdesc,
-                                                char relkind, bool relhasoids, bool istemp,
+                                                char relkind,
+                                                bool relhasoids,
                                                 bool allow_system_table_mods);
 
 extern void heap_drop_with_catalog(Oid rid, bool allow_system_table_mods);
index 86f0824e84ad99e042586ab11cc39ab6683c0251..70e33f88fd3746b33a711458ac66abdbad7794fb 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: index.h,v 1.46 2002/03/26 19:16:25 tgl Exp $
+ * $Id: index.h,v 1.47 2002/03/31 06:26:32 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -30,11 +30,10 @@ typedef void (*IndexBuildCallback) (Relation index,
 
 
 extern Oid index_create(Oid heapRelationId,
-                        char *indexRelationName,
+                        const char *indexRelationName,
                         IndexInfo *indexInfo,
                         Oid accessMethodObjectId,
                         Oid *classObjectId,
-                        bool istemp,
                         bool primary,
                         bool allow_system_table_mods);
 
index 242050de490a8305ffc8aad037628effd2fa3f06..aed33a10b31d8d4fdcb1040e3f742a7f081686d2 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: namespace.h,v 1.3 2002/03/30 01:02:42 tgl Exp $
+ * $Id: namespace.h,v 1.4 2002/03/31 06:26:32 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -29,4 +29,6 @@ extern Oid    QualifiedNameGetCreationNamespace(List *names, char **objname_p);
 
 extern RangeVar *makeRangeVarFromNameList(List *names);
 
+extern bool isTempNamespace(Oid namespaceId);
+
 #endif   /* NAMESPACE_H */
index 607b74132e4b790d9fbf6fad5f815bdcb86405de..fceee09fcb774689716524dd44cfc9f3b08c6bdc 100644 (file)
@@ -8,7 +8,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: pg_namespace.h,v 1.2 2002/03/30 01:02:42 tgl Exp $
+ * $Id: pg_namespace.h,v 1.3 2002/03/31 06:26:32 tgl Exp $
  *
  * NOTES
  *       the genbki.sh script reads this file and generates .bki
@@ -69,7 +69,7 @@ typedef FormData_pg_namespace *Form_pg_namespace;
 DATA(insert OID = 11 ( "pg_catalog" PGUID "{=r}" ));
 DESCR("System catalog namespace");
 #define PG_CATALOG_NAMESPACE 11
-DATA(insert OID = 99 ( "pg_toast" PGUID "{=r}" ));
+DATA(insert OID = 99 ( "pg_toast" PGUID "{=}" ));
 DESCR("Reserved namespace for TOAST tables");
 #define PG_TOAST_NAMESPACE 99
 DATA(insert OID = 2071 ( "pg_public" PGUID "{=rw}" ));
@@ -80,6 +80,6 @@ DESCR("Standard public namespace");
 /*
  * prototypes for functions in pg_namespace.c
  */
-extern Oid     NamespaceCreate(const char *nspName);
+extern Oid     NamespaceCreate(const char *nspName, int32 ownerSysId);
 
 #endif   /* PG_NAMESPACE_H */
index 16398b4c08f977754d6db428407f47dac753b471..4a3fbd1eb1534ed3bb3ba959399ccc486c1ee125 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: rel.h,v 1.57 2002/03/26 19:16:58 tgl Exp $
+ * $Id: rel.h,v 1.58 2002/03/31 06:26:32 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -238,49 +238,15 @@ typedef Relation *RelationPtr;
  */
 #define RelationGetIndexStrategy(relation) ((relation)->rd_istrat)
 
-/*
- * Handle temp relations
- */
-#define PG_TEMP_REL_PREFIX "pg_temp"
-#define PG_TEMP_REL_PREFIX_LEN 7
-
-#define is_temp_relname(relname) \
-               (strncmp(relname, PG_TEMP_REL_PREFIX, PG_TEMP_REL_PREFIX_LEN) == 0)
-
-/*
- * RelationGetPhysicalRelationName
- *
- *       Returns the rel's physical name, ie, the name appearing in pg_class.
- *
- * While this name is unique across all rels in the database, it is not
- * necessarily useful for accessing the rel, since a temp table of the
- * same name might mask the rel.  It is useful mainly for determining if
- * the rel is a shared system rel or not.
- *
- * The macro is rather unfortunately named, since the pg_class name no longer
- * has anything to do with the file name used for physical storage of the rel.
- */
-#define RelationGetPhysicalRelationName(relation) \
-       (NameStr((relation)->rd_rel->relname))
-
 /*
  * RelationGetRelationName
  *
- *       Returns the relation's logical name (as seen by the user).
+ *       Returns the rel's name.
  *
- * If the rel is a temp rel, the temp name will be returned.  Therefore,
- * this name is not unique.  But it is the name to use in heap_openr(),
- * for example.
+ * Note that the name is only unique within the containing namespace.
  */
 #define RelationGetRelationName(relation) \
-(\
-       is_temp_relname(RelationGetPhysicalRelationName(relation)) \
-       ? \
-               get_temp_rel_by_physicalname( \
-                       RelationGetPhysicalRelationName(relation)) \
-       : \
-               RelationGetPhysicalRelationName(relation) \
-)
+       (NameStr((relation)->rd_rel->relname))
 
 /*
  * RelationGetNamespace
@@ -290,7 +256,4 @@ typedef Relation *RelationPtr;
 #define RelationGetNamespace(relation) \
        ((relation)->rd_rel->relnamespace)
 
-/* added to prevent circular dependency.  bjm 1999/11/15 */
-extern char *get_temp_rel_by_physicalname(const char *relname);
-
 #endif   /* REL_H */
diff --git a/src/include/utils/temprel.h b/src/include/utils/temprel.h
deleted file mode 100644 (file)
index a8a6a00..0000000
+++ /dev/null
@@ -1,33 +0,0 @@
-/*-------------------------------------------------------------------------
- *
- * temprel.h
- *       Temporary relation functions
- *
- *
- * Portions Copyright (c) 1996-2001, PostgreSQL Global Development Group
- * Portions Copyright (c) 1994, Regents of the University of California
- *
- * $Id: temprel.h,v 1.20 2001/11/05 17:46:36 momjian Exp $
- *
- *-------------------------------------------------------------------------
- */
-#ifndef TEMPREL_H
-#define TEMPREL_H
-
-#include "access/htup.h"
-
-extern void create_temp_relation(const char *relname,
-                                        HeapTuple pg_class_tuple);
-extern void remove_temp_rel_by_relid(Oid relid);
-extern bool rename_temp_relation(const char *oldname,
-                                        const char *newname);
-
-extern void remove_all_temp_relations(void);
-extern void AtEOXact_temp_relations(bool isCommit);
-
-extern char *get_temp_rel_by_username(const char *user_relname);
-extern char *get_temp_rel_by_physicalname(const char *relname);
-
-#define is_temp_rel_name(relname) (get_temp_rel_by_username(relname) != NULL)
-
-#endif   /* TEMPREL_H */