Create 'default_tablespace' GUC variable that supplies a TABLESPACE
authorTom Lane
Fri, 5 Nov 2004 19:17:13 +0000 (19:17 +0000)
committerTom Lane
Fri, 5 Nov 2004 19:17:13 +0000 (19:17 +0000)
clause implicitly whenever one is not given explicitly.  Remove concept
of a schema having an associated tablespace, and simplify the rules for
selecting a default tablespace for a table or index.  It's now just
(a) explicit TABLESPACE clause; (b) default_tablespace if that's not an
empty string; (c) database's default.  This will allow pg_dump to use
SET commands instead of tablespace clauses to determine object locations
(but I didn't actually make it do so).  All per recent discussions.

37 files changed:
contrib/oid2name/oid2name.c
doc/src/sgml/catalogs.sgml
doc/src/sgml/manage-ag.sgml
doc/src/sgml/ref/create_index.sgml
doc/src/sgml/ref/create_schema.sgml
doc/src/sgml/ref/create_table.sgml
doc/src/sgml/ref/create_tablespace.sgml
doc/src/sgml/ref/grant.sgml
doc/src/sgml/ref/revoke.sgml
doc/src/sgml/release.sgml
doc/src/sgml/runtime.sgml
src/backend/catalog/namespace.c
src/backend/catalog/pg_namespace.c
src/backend/commands/indexcmds.c
src/backend/commands/schemacmds.c
src/backend/commands/tablecmds.c
src/backend/commands/tablespace.c
src/backend/nodes/copyfuncs.c
src/backend/nodes/equalfuncs.c
src/backend/parser/gram.y
src/backend/utils/adt/ruleutils.c
src/backend/utils/cache/lsyscache.c
src/backend/utils/misc/guc.c
src/backend/utils/misc/postgresql.conf.sample
src/bin/pg_dump/pg_dump.c
src/bin/pg_dump/pg_dump.h
src/bin/psql/describe.c
src/bin/psql/tab-complete.c
src/include/catalog/catversion.h
src/include/catalog/pg_namespace.h
src/include/commands/tablespace.h
src/include/nodes/parsenodes.h
src/include/utils/guc.h
src/include/utils/lsyscache.h
src/interfaces/ecpg/preproc/preproc.y
src/test/regress/input/tablespace.source
src/test/regress/output/tablespace.source

index 7186a7abd7fecb18477a6ba6fbc5ddc8f52d4ed7..05f73ab1f58e096103347b9b1f3be07534785eb4 100644 (file)
@@ -367,7 +367,8 @@ sql_exec_dumpalldbs(PGconn *conn, struct options *opts)
    char        todo[1024];
 
    /* get the oid and database name from the system pg_database table */
-   snprintf(todo, 1024, "SELECT d.oid AS \"Oid\", datname AS \"Database Name\", "
+   snprintf(todo, sizeof(todo),
+            "SELECT d.oid AS \"Oid\", datname AS \"Database Name\", "
             "spcname AS \"Tablespace\" FROM pg_database d JOIN pg_tablespace t ON "
             "(dattablespace = t.oid) ORDER BY 2");
 
@@ -383,7 +384,7 @@ sql_exec_dumpalltables(PGconn *conn, struct options *opts)
    char        todo[1024];
    char       *addfields = ",c.oid AS \"Oid\", nspname AS \"Schema\", spcname as \"Tablespace\" ";
 
-   snprintf(todo, 1024,
+   snprintf(todo, sizeof(todo),
             "SELECT relfilenode as \"Filenode\", relname as \"Table Name\" %s "
             "FROM pg_class c "
             "  LEFT JOIN pg_catalog.pg_namespace n ON n.oid = c.relnamespace "
@@ -393,8 +394,7 @@ sql_exec_dumpalltables(PGconn *conn, struct options *opts)
             "  %s"
             "      t.oid = CASE"
             "          WHEN reltablespace <> 0 THEN reltablespace"
-            "          WHEN n.nsptablespace <> 0 THEN nsptablespace"
-            "          WHEN d.dattablespace <> 0 THEN dattablespace"
+            "          ELSE dattablespace"
             "      END "
             "ORDER BY relname",
             opts->extended ? addfields : "",
@@ -451,7 +451,7 @@ sql_exec_searchtables(PGconn *conn, struct options *opts)
 
    /* now build the query */
    todo = (char *) myalloc(650 + strlen(qualifiers));
-   snprintf(todo, 1024,
+   snprintf(todo, 650 + strlen(qualifiers),
             "SELECT relfilenode as \"Filenode\", relname as \"Table Name\" %s\n"
             "FROM pg_class c \n"
             "  LEFT JOIN pg_catalog.pg_namespace n ON n.oid = c.relnamespace \n"
@@ -460,8 +460,7 @@ sql_exec_searchtables(PGconn *conn, struct options *opts)
             "WHERE relkind IN ('r', 'i', 'S', 't') AND \n"
             "      t.oid = CASE\n"
             "          WHEN reltablespace <> 0 THEN reltablespace\n"
-            "          WHEN n.nsptablespace <> 0 THEN nsptablespace\n"
-            "          WHEN d.dattablespace <> 0 THEN dattablespace\n"
+            "          ELSE dattablespace\n"
             "      END AND \n"
             "  (%s) \n"
             "ORDER BY relname\n",
@@ -478,7 +477,8 @@ sql_exec_dumpalltbspc(PGconn *conn, struct options *opts)
 {
    char    todo[1024];
 
-   snprintf(todo, 1024, "SELECT oid AS \"Oid\", spcname as \"Tablespace Name\"\n"
+   snprintf(todo, sizeof(todo),
+            "SELECT oid AS \"Oid\", spcname as \"Tablespace Name\"\n"
             "FROM pg_tablespace");
 
    sql_exec(conn, todo, opts->quiet);
index aef5a751bb6bcb7d7f702b5d43000ad1f1ca9627..e7cd4ec7e4a26bae1d3eed313f35c27eaf17fc13 100644 (file)
@@ -1,6 +1,6 @@
 
 
 
       Owner of the namespace
      
 
-     
-      nsptablespace
-      oid
-      pg_tablespace.oid
-      
-       The default tablespace in which to place relations created in this
-       namespace.  If zero, the database's default tablespace is implied.
-       (Changing this does not affect pre-existing relations.)
-      
-     
-
      
       nspacl
       aclitem[]
index 5db8e939e4d86202c773c877f7579e16788f6301..0237e026b19d80d62da64c2aff313f14af550a77 100644 (file)
@@ -1,5 +1,5 @@
 
 
 
@@ -395,7 +395,7 @@ CREATE TABLESPACE fastspace LOCATION '/mnt/sda1/postgresql/data';
    
 
    
-    Databases, schemas, tables, and indexes can all be assigned to
+    Tables, indexes, and entire databases can be assigned to
     particular tablespaces. To do so, a user with the CREATE
     privilege on a given tablespace must pass the tablespace name as a 
     parameter to the relevant command. For example, the following creates 
@@ -406,37 +406,26 @@ CREATE TABLE foo(i int) TABLESPACE space1;
    
 
    
-    The tablespace associated with a database is used to store the system
-    catalogs of that database, as well as any temporary files created by
-    server processes using that database.  Furthermore, it is the default
-    tablespace selected for any objects created within the database, if
-    no specific TABLESPACE clause is given when those objects
-    are created.  If a database is created without specifying a tablespace
-    for it, it uses the same tablespace as the template database it is copied
-    from.
-   
-
-   
-    A schema does not in itself occupy any storage (other than a
-    system catalog entry), so assigning a schema to a tablespace does
-    not in itself do anything.  What this actually does is to set a
-    default tablespace for tables later created within the schema.  If
-    no tablespace is mentioned when creating a schema, it inherits its
-    default tablespace from the current database.
-   
-
-   
-    The default tablespace for an index is the tablespace associated
-    with the table the index is on.
+    Alternatively, use the  parameter:
+
+SET default_tablespace = space1;
+CREATE TABLE foo(i int);
+
+    When default_tablespace is set to anything but an empty
+    string, it supplies an implicit TABLESPACE clause for
+    CREATE TABLE and CREATE INDEX commands that
+    do not have an explicit one.
    
 
    
-    Another way to state the above rules is that when a schema, table, or index
-    is created without specifying a tablespace, the object
-    inherits its logical parent's tablespace. A schema will be created in the
-    current database's tablespace; a table will be created in the
-    tablespace of the schema it is being created in; an index will be created
-    in the tablespace of the table underlying the index.
+    The tablespace associated with a database is used to store the system
+    catalogs of that database, as well as any temporary files created by
+    server processes using that database.  Furthermore, it is the default
+    tablespace selected for tables and indexes created within the database,
+    if no TABLESPACE clause is given (either explicitly or via
+    default_tablespace) when the objects are created.
+    If a database is created without specifying a tablespace for it,
+    it uses the same tablespace as the template database it is copied from.
    
 
    
@@ -444,9 +433,9 @@ CREATE TABLE foo(i int) TABLESPACE space1;
     pg_global tablespace is used for shared system catalogs. The
     pg_default tablespace is the default tablespace of the
     template1 and template0 databases (and, therefore,
-    will be the default tablespace for everything else as well, unless
-    explicit TABLESPACE clauses are used somewhere along the
-    line).
+    will be the default tablespace for other databases as well, unless
+    overridden by a TABLESPACE clause in CREATE
+    DATABASE).
    
 
    
index c45df0c5be058933398aa53c32f84f9aab3e7109..7f53ad24b5a64a582f71a9eb4cd757ed6d184452 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -186,7 +186,9 @@ CREATE [ UNIQUE ] INDEX name ON 
       
        
         The tablespace in which to create the index.  If not specified,
-        the tablespace of the parent table is used.
+         is used, or the database's
+        default tablespace if default_tablespace is an empty
+        string.
        
       
      
index 188d53b68eda03883903fc038fc4c5deb991e012..195ab01042379c95125119159181f1ca6ec42e62 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -20,8 +20,8 @@ PostgreSQL documentation
 
  
 
-CREATE SCHEMA schemaname [ AUTHORIZATION username ] [ TABLESPACE tablespace ] [ schema_element [ ... ] ]
-CREATE SCHEMA AUTHORIZATION username [ TABLESPACE tablespace ] [ schema_element [ ... ] ]
+CREATE SCHEMA schemaname [ AUTHORIZATION username ] [ schema_element [ ... ] ]
+CREATE SCHEMA AUTHORIZATION username [ schema_element [ ... ] ]
 
  
 
@@ -82,17 +82,6 @@ CREATE SCHEMA AUTHORIZATION username
       
      
 
-     
-      tablespace
-      
-       
-        The name of the tablespace that is to be the default tablespace
-        for all new objects created in the schema. If not supplied, the schema
-        will inherit the default tablespace of the database.
-       
-      
-     
-
      
       schema_element
       
@@ -116,9 +105,7 @@ CREATE SCHEMA AUTHORIZATION username
   
    To create a schema, the invoking user must have the
    CREATE privilege for the current database.
-   Also, the TABLESPACE option requires having
-   CREATE privilege for the specified tablespace.
-   (Of course, superusers bypass these checks.)
+   (Of course, superusers bypass this check.)
   
  
 
@@ -161,15 +148,6 @@ CREATE VIEW hollywood.winners AS
 
   
 
-  
-   Create a schema sales whose tables and indexes
-   will be stored in the tablespace mirrorspace by default:
-
-
-CREATE SCHEMA sales TABLESPACE mirrorspace;
-
-  
-
  
 
  
@@ -206,7 +184,6 @@ CREATE SCHEMA sales TABLESPACE mirrorspace;
   
    
    
-    
  
  
 
index e70f0c3dfd00478bb8451934f2d501c893f19e84..8b18d837c96af38529f21d3da33ce76abc0bcdea 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -603,8 +603,11 @@ and table_constraint is:
     
      
       The tablespace is the name
-      of the tablespace in which the new table is to be created. If not
-      supplied, the default tablespace of the table's schema will be used.
+      of the tablespace in which the new table is to be created.
+      If not specified,
+       is used, or the database's
+      default tablespace if default_tablespace is an empty
+      string.
      
     
    
@@ -615,8 +618,11 @@ and table_constraint is:
      
       This clause allows selection of the tablespace in which the index
       associated with a UNIQUE or PRIMARY
-      KEY constraint will be created.  If not supplied, the index
-      will be created in the same tablespace as the table.
+      KEY constraint will be created.
+      If not specified,
+       is used, or the database's
+      default tablespace if default_tablespace is an empty
+      string.
      
     
    
index 76161eb1aed6cf0d1b4f2e5263efc5713bbcf100..87a44d00d0b63890ff249e907399ef486fe6fd53 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -41,8 +41,8 @@ CREATE TABLESPACE tablespacename [
 
   
    A user with appropriate privileges can pass
-   tablespacename to CREATE
-   DATABASE, CREATE SCHEMACREATE TABLE,
+   tablespacename to
+   CREATE DATABASECREATE TABLE,
    CREATE INDEX or ADD CONSTRAINT to have the data
    files for these objects stored within the specified tablespace.
   
@@ -130,7 +130,6 @@ CREATE TABLESPACE indexspace OWNER genevieve LOCATION '/data/indexes';
 
   
    
-   
    
    
    
index d6a6ef94b6f0b73427c8961c2a73f85b09ab5377..97854d5528179e715c07cf1a2dd2b3968b0a03b8 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -29,10 +29,6 @@ GRANT { { CREATE | TEMPORARY | TEMP } [,...] | ALL [ PRIVILEGES ] }
     ON DATABASE dbname [, ...]
     TO { username | GROUP groupname | PUBLIC } [, ...] [ WITH GRANT OPTION ]
 
-GRANT { CREATE | ALL [ PRIVILEGES ] }
-    ON TABLESPACE tablespacename [, ...]
-    TO { username | GROUP groupname | PUBLIC } [, ...] [ WITH GRANT OPTION ]
-
 GRANT { EXECUTE | ALL [ PRIVILEGES ] }
     ON FUNCTION funcname ([type, ...]) [, ...]
     TO { username | GROUP groupname | PUBLIC } [, ...] [ WITH GRANT OPTION ]
@@ -44,6 +40,10 @@ GRANT { USAGE | ALL [ PRIVILEGES ] }
 GRANT { { CREATE | USAGE } [,...] | ALL [ PRIVILEGES ] }
     ON SCHEMA schemaname [, ...]
     TO { username | GROUP groupname | PUBLIC } [, ...] [ WITH GRANT OPTION ]
+
+GRANT { CREATE | ALL [ PRIVILEGES ] }
+    ON TABLESPACE tablespacename [, ...]
+    TO { username | GROUP groupname | PUBLIC } [, ...] [ WITH GRANT OPTION ]
 
  
 
@@ -52,8 +52,8 @@ GRANT { { CREATE | USAGE } [,...] | ALL [ PRIVILEGES ] }
 
   
    The GRANT command gives specific privileges on
-   an object (table, view, sequence, database, tablespace, function,
-   procedural language, or schema) to
+   an object (table, view, sequence, database, function,
+   procedural language, schema, or tablespace) to
    one or more users or groups of users.  These privileges are added
    to those already granted, if any.
   
@@ -188,17 +188,17 @@ GRANT { { CREATE | USAGE } [,...] | ALL [ PRIVILEGES ] }
       
        For databases, allows new schemas to be created within the database.
       
-      
-       For tablespaces, allows tables to be created within the tablespace,
-       and allows databases and schemas to be created that have the tablespace
-       as their default tablespace.  (Note that revoking this privilege
-       will not alter the behavior of existing databases and schemas.)
-      
       
        For schemas, allows new objects to be created within the schema.
        To rename an existing object, you must own the object and
        have this privilege for the containing schema.
       
+      
+       For tablespaces, allows tables and indexes to be created within the
+       tablespace, and allows databases to be created that have the tablespace
+       as their default tablespace.  (Note that revoking this privilege
+       will not alter the placement of existing objects.)
+      
      
     
 
index c6cd587ade71641a99a501df6e5607e10900a6cf..5f94537e26e3d24646487bcb7156364a9543370f 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -33,12 +33,6 @@ REVOKE [ GRANT OPTION FOR ]
     FROM { username | GROUP groupname | PUBLIC } [, ...]
     [ CASCADE | RESTRICT ]
 
-REVOKE [ GRANT OPTION FOR ]
-    { CREATE | ALL [ PRIVILEGES ] }
-    ON TABLESPACE tablespacename [, ...]
-    FROM { username | GROUP groupname | PUBLIC } [, ...]
-    [ CASCADE | RESTRICT ]
-
 REVOKE [ GRANT OPTION FOR ]
     { EXECUTE | ALL [ PRIVILEGES ] }
     ON FUNCTION funcname ([type, ...]) [, ...]
@@ -56,6 +50,12 @@ REVOKE [ GRANT OPTION FOR ]
     ON SCHEMA schemaname [, ...]
     FROM { username | GROUP groupname | PUBLIC } [, ...]
     [ CASCADE | RESTRICT ]
+
+REVOKE [ GRANT OPTION FOR ]
+    { CREATE | ALL [ PRIVILEGES ] }
+    ON TABLESPACE tablespacename [, ...]
+    FROM { username | GROUP groupname | PUBLIC } [, ...]
+    [ CASCADE | RESTRICT ]
 
  
 
index 466754b11ffd70b93c3ec39d8922fc9a6b84a22f..1ec93a215f11c26f29138044238c178f015b76b1 100644 (file)
@@ -1,5 +1,5 @@
 
 
 
@@ -96,8 +96,8 @@ $PostgreSQL: pgsql/doc/src/sgml/release.sgml,v 1.303 2004/10/24 22:43:56 tgl Exp
       
        
         Tablespaces allow administrators to select the file systems
-        used for storage of databases, schemas, tables, or
-        indexes. This improves performance and control over disk space
+        used for storage of tables, indexes, and entire databases.
+        This improves performance and control over disk space
         usage. Prior releases used initlocation and
         manual symlink management for such tasks.
        
index be2ced298d3e8d9fabbdd2ff557de585befe468a..91cdec32284652677a51116832102f0d6bc25050 100644 (file)
@@ -1,5 +1,5 @@
 
 
 
@@ -2720,6 +2720,32 @@ archive_command = 'copy "%p" /mnt/server/archivedir/"%f"'  # Win32
       
      
 
+     
+      default_tablespace (string)
+      default_tablespace
+      tablespacedefault
+      
+       
+        This variable specifies the default tablespace in which to create
+        objects (tables and indexes) when a CREATE command does
+        not explicitly specify a tablespace.
+       
+
+       
+        The value is either the name of a tablespace, or an empty string
+        to specify using the default tablespace of the current database.
+        If the value does not match the name of any existing tablespace,
+        PostgreSQL will automatically use the default
+        tablespace of the current database.
+       
+
+       
+        For more information on tablespaces,
+        see .
+       
+      
+     
+
      
       check_function_bodies (boolean)
       
index f722b031e663a85c16d3ddf098e878805c9bd246..b83b62659d8a6a0b33d2dbb994375f4c81658e50 100644 (file)
@@ -13,7 +13,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/catalog/namespace.c,v 1.71 2004/09/16 16:58:27 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/catalog/namespace.c,v 1.72 2004/11/05 19:15:54 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1675,7 +1675,7 @@ InitTempTableNamespace(void)
         * that access the temp namespace for my own backend skip
         * permissions checks on it.
         */
-       namespaceId = NamespaceCreate(namespaceName, BOOTSTRAP_USESYSID, 0);
+       namespaceId = NamespaceCreate(namespaceName, BOOTSTRAP_USESYSID);
        /* Advance command counter to make namespace visible */
        CommandCounterIncrement();
    }
index c6ac46d754029bba6c70c74900895afc481301de..886392fa4ce18faa913cd4cc4e5dd28da742075c 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/catalog/pg_namespace.c,v 1.10 2004/08/29 04:12:29 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/catalog/pg_namespace.c,v 1.11 2004/11/05 19:15:54 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -27,7 +27,7 @@
  * ---------------
  */
 Oid
-NamespaceCreate(const char *nspName, int32 ownerSysId, Oid nspTablespace)
+NamespaceCreate(const char *nspName, int32 ownerSysId)
 {
    Relation    nspdesc;
    HeapTuple   tup;
@@ -59,7 +59,6 @@ NamespaceCreate(const char *nspName, int32 ownerSysId, Oid nspTablespace)
    namestrcpy(&nname, nspName);
    values[Anum_pg_namespace_nspname - 1] = NameGetDatum(&nname);
    values[Anum_pg_namespace_nspowner - 1] = Int32GetDatum(ownerSysId);
-   values[Anum_pg_namespace_nsptablespace - 1] = Int32GetDatum(nspTablespace);
    nulls[Anum_pg_namespace_nspacl - 1] = 'n';
 
    nspdesc = heap_openr(NamespaceRelationName, RowExclusiveLock);
index bdbf8708b1362df2ad916ab01c7c9d992e68fb92..b3f80470c0434735ef985af834dfe3a1c27cb335 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/commands/indexcmds.c,v 1.126 2004/09/13 20:06:29 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/commands/indexcmds.c,v 1.127 2004/11/05 19:15:57 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -24,6 +24,7 @@
 #include "catalog/namespace.h"
 #include "catalog/pg_opclass.h"
 #include "catalog/pg_proc.h"
+#include "catalog/pg_tablespace.h"
 #include "commands/dbcommands.h"
 #include "commands/defrem.h"
 #include "commands/tablecmds.h"
@@ -66,7 +67,7 @@ static bool relationHasPrimaryKey(Relation rel);
  *     that a nonconflicting default name should be picked.
  * 'accessMethodName': name of the AM to use.
  * 'tableSpaceName': name of the tablespace to create the index in.
- *     NULL specifies using the same tablespace as the parent relation.
+ *     NULL specifies using the appropriate default.
  * 'attributeList': a list of IndexElem specifying columns and expressions
  *     to index on.
  * 'predicate': the partial-index condition, or NULL if none.
@@ -157,31 +158,44 @@ DefineIndex(RangeVar *heapRelation,
                           get_namespace_name(namespaceId));
    }
 
-   /* Determine tablespace to use */
+   /*
+    * Select tablespace to use.  If not specified, use default_tablespace
+    * (which may in turn default to database's default).
+    */
    if (tableSpaceName)
    {
-       AclResult   aclresult;
-
        tablespaceId = get_tablespace_oid(tableSpaceName);
        if (!OidIsValid(tablespaceId))
            ereport(ERROR,
                    (errcode(ERRCODE_UNDEFINED_OBJECT),
                     errmsg("tablespace \"%s\" does not exist",
                            tableSpaceName)));
-       /* check permissions */
+   }
+   else
+   {
+       tablespaceId = GetDefaultTablespace();
+       /* note InvalidOid is OK in this case */
+   }
+
+   /* Check permissions except when using database's default */
+   if (OidIsValid(tablespaceId))
+   {
+       AclResult   aclresult;
+
        aclresult = pg_tablespace_aclcheck(tablespaceId, GetUserId(),
                                           ACL_CREATE);
        if (aclresult != ACLCHECK_OK)
            aclcheck_error(aclresult, ACL_KIND_TABLESPACE,
-                          tableSpaceName);
-   }
-   else
-   {
-       /* Use the parent rel's tablespace */
-       tablespaceId = get_rel_tablespace(relationId);
-       /* Note there is no additional permission check in this path */
+                          get_tablespace_name(tablespaceId));
    }
 
+   /*
+    * Force shared indexes into the pg_global tablespace.  This is a bit of
+    * a hack but seems simpler than marking them in the BKI commands.
+    */
+   if (rel->rd_rel->relisshared)
+       tablespaceId = GLOBALTABLESPACE_OID;
+
    /*
     * Select name for index if caller didn't specify
     */
index b4e1cd77983680673339c170805a64eb3d7f9a37..fc2410bacc11117f0f056e5414cb01e7b062a849 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/commands/schemacmds.c,v 1.25 2004/09/02 00:22:16 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/commands/schemacmds.c,v 1.26 2004/11/05 19:15:57 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -23,7 +23,6 @@
 #include "catalog/pg_namespace.h"
 #include "commands/dbcommands.h"
 #include "commands/schemacmds.h"
-#include "commands/tablespace.h"
 #include "miscadmin.h"
 #include "parser/analyze.h"
 #include "tcop/utility.h"
@@ -42,7 +41,6 @@ CreateSchemaCommand(CreateSchemaStmt *stmt)
    const char *schemaName = stmt->schemaname;
    const char *authId = stmt->authid;
    Oid         namespaceId;
-   Oid         tablespaceId;
    List       *parsetree_list;
    ListCell   *parsetree_item;
    const char *owner_name;
@@ -102,35 +100,8 @@ CreateSchemaCommand(CreateSchemaStmt *stmt)
                 errmsg("unacceptable schema name \"%s\"", schemaName),
        errdetail("The prefix \"pg_\" is reserved for system schemas.")));
 
-   /*
-    * Select default tablespace for schema.  If not given, use zero which
-    * implies the database's default tablespace.
-    */
-   if (stmt->tablespacename)
-   {
-       AclResult   aclresult;
-
-       tablespaceId = get_tablespace_oid(stmt->tablespacename);
-       if (!OidIsValid(tablespaceId))
-           ereport(ERROR,
-                   (errcode(ERRCODE_UNDEFINED_OBJECT),
-                    errmsg("tablespace \"%s\" does not exist",
-                           stmt->tablespacename)));
-       /* check permissions */
-       aclresult = pg_tablespace_aclcheck(tablespaceId, GetUserId(),
-                                          ACL_CREATE);
-       if (aclresult != ACLCHECK_OK)
-           aclcheck_error(aclresult, ACL_KIND_TABLESPACE,
-                          stmt->tablespacename);
-   }
-   else
-   {
-       tablespaceId = InvalidOid;
-       /* note there is no permission check in this path */
-   }
-
    /* Create the schema's namespace */
-   namespaceId = NamespaceCreate(schemaName, owner_userid, tablespaceId);
+   namespaceId = NamespaceCreate(schemaName, owner_userid);
 
    /* Advance cmd counter to make the namespace visible */
    CommandCounterIncrement();
index 4593327d35d1749787cb2a7da5b7817a35960cda..e4001f0102a48a37a18927cdb14a04752669972d 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/commands/tablecmds.c,v 1.138 2004/10/30 20:52:56 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/commands/tablecmds.c,v 1.139 2004/11/05 19:15:57 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -168,7 +168,6 @@ static void StoreCatalogInheritance(Oid relationId, List *supers);
 static int findAttrByName(const char *attributeName, List *schema);
 static void setRelhassubclassInRelation(Oid relationId, bool relhassubclass);
 static bool needs_toast_table(Relation rel);
-static void check_tablespace_exists(Oid tablespaceId, Oid namespaceId);
 static int transformColumnNameList(Oid relId, List *colList,
                        int16 *attnums, Oid *atttypids);
 static int transformFkeyGetPrimaryKey(Relation pkrel, Oid *indexOid,
@@ -313,34 +312,34 @@ DefineRelation(CreateStmt *stmt, char relkind)
    }
 
    /*
-    * Select tablespace to use.  If not specified, use containing
-    * schema's default tablespace (which may in turn default to
-    * database's default).
+    * Select tablespace to use.  If not specified, use default_tablespace
+    * (which may in turn default to database's default).
     */
    if (stmt->tablespacename)
    {
-       AclResult   aclresult;
-
        tablespaceId = get_tablespace_oid(stmt->tablespacename);
        if (!OidIsValid(tablespaceId))
            ereport(ERROR,
                    (errcode(ERRCODE_UNDEFINED_OBJECT),
                     errmsg("tablespace \"%s\" does not exist",
                            stmt->tablespacename)));
-       /* check permissions */
+   }
+   else
+   {
+       tablespaceId = GetDefaultTablespace();
+       /* note InvalidOid is OK in this case */
+   }
+
+   /* Check permissions except when using database's default */
+   if (OidIsValid(tablespaceId))
+   {
+       AclResult   aclresult;
+
        aclresult = pg_tablespace_aclcheck(tablespaceId, GetUserId(),
                                           ACL_CREATE);
        if (aclresult != ACLCHECK_OK)
            aclcheck_error(aclresult, ACL_KIND_TABLESPACE,
-                          stmt->tablespacename);
-   }
-   else
-   {
-       tablespaceId = get_namespace_tablespace(namespaceId);
-       /* note no permission check on tablespace in this case */
-       /* check to see that schema's tablespace still exists */
-       if (OidIsValid(tablespaceId))
-           check_tablespace_exists(tablespaceId, namespaceId);
+                          get_tablespace_name(tablespaceId));
    }
 
    /*
@@ -5890,42 +5889,6 @@ needs_toast_table(Relation rel)
    return (tuple_length > TOAST_TUPLE_THRESHOLD);
 }
 
-/*
- * Verify that a schema's tablespace still exists
- *
- * We need this because DROP TABLESPACE cannot check whether the target
- * tablespace is the default tablespace for any schemas.  (It could check
- * in the current database, but that doesn't seem very helpful.)  Subsequent
- * attempts to create tables in that tablespace will fail.  This code just
- * exists to ensure that we give a helpful error message.
- */
-static void
-check_tablespace_exists(Oid tablespaceId, Oid namespaceId)
-{
-   Relation    pg_tablespace;
-   ScanKeyData entry[1];
-   HeapScanDesc scan;
-   HeapTuple   tuple;
-
-   /* There's no syscache for pg_tablespace, so must look the hard way */
-   pg_tablespace = heap_openr(TableSpaceRelationName, AccessShareLock);
-   ScanKeyInit(&entry[0],
-               ObjectIdAttributeNumber,
-               BTEqualStrategyNumber, F_OIDEQ,
-               ObjectIdGetDatum(tablespaceId));
-   scan = heap_beginscan(pg_tablespace, SnapshotNow, 1, entry);
-   tuple = heap_getnext(scan, ForwardScanDirection);
-   if (!HeapTupleIsValid(tuple))
-       ereport(ERROR,
-               (errcode(ERRCODE_UNDEFINED_OBJECT),
-                errmsg("tablespace with OID %u does not exist",
-                       tablespaceId),
-                errdetail("The default tablespace for schema \"%s\" has been dropped.",
-                          get_namespace_name(namespaceId))));
-   heap_endscan(scan);
-   heap_close(pg_tablespace, AccessShareLock);
-}
-
 
 /*
  * This code supports
index 55f086d6609a42909e877fab0b62c3a629f5d53b..4dd1120c9d4ca8c78baff024d660028c547d0339 100644 (file)
@@ -3,7 +3,6 @@
  * tablespace.c
  *   Commands to manipulate table spaces
  *
- *
  * Tablespaces in PostgreSQL are designed to allow users to determine
  * where the data file(s) for a given database object reside on the file
  * system.
  *         $PGDATA/global/relfilenode
  *         $PGDATA/base/dboid/relfilenode
  *
- * The implementation is designed to be backwards compatible. For this reason
- * (and also as a feature unto itself) when a user creates an object without
- * specifying a tablespace, we look at the object's parent and place
- * the object in the parent's tablespace. The hierarchy is as follows:
- *         database > schema > table > index
- *
  * To allow CREATE DATABASE to give a new database a default tablespace
  * that's different from the template database's default, we make the
  * provision that a zero in pg_class.reltablespace means the database's
  * default tablespace. Without this, CREATE DATABASE would have to go in
- * and munge the system catalogs of the new database.  This special meaning
- * of zero also applies in pg_namespace.nsptablespace.
+ * and munge the system catalogs of the new database.
  *
  *
  * Portions Copyright (c) 1996-2004, PostgreSQL Global Development Group
@@ -45,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/commands/tablespace.c,v 1.13 2004/11/05 17:11:05 petere Exp $
+ *   $PostgreSQL: pgsql/src/backend/commands/tablespace.c,v 1.14 2004/11/05 19:15:57 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
 #include "utils/acl.h"
 #include "utils/builtins.h"
 #include "utils/fmgroids.h"
+#include "utils/guc.h"
 #include "utils/lsyscache.h"
 #include "utils/syscache.h"
 
 
+/* GUC variable */
+char   *default_tablespace = NULL;
+
+
 static bool remove_tablespace_directories(Oid tablespaceoid, bool redo);
 static void set_short_version(const char *path);
 
@@ -725,77 +722,6 @@ directory_is_empty(const char *path)
    return true;
 }
 
-/*
- * get_tablespace_oid - given a tablespace name, look up the OID
- *
- * Returns InvalidOid if tablespace name not found.
- */
-Oid
-get_tablespace_oid(const char *tablespacename)
-{
-   Oid         result;
-   Relation    rel;
-   HeapScanDesc scandesc;
-   HeapTuple   tuple;
-   ScanKeyData entry[1];
-
-   /* Search pg_tablespace */
-   rel = heap_openr(TableSpaceRelationName, AccessShareLock);
-
-   ScanKeyInit(&entry[0],
-               Anum_pg_tablespace_spcname,
-               BTEqualStrategyNumber, F_NAMEEQ,
-               CStringGetDatum(tablespacename));
-   scandesc = heap_beginscan(rel, SnapshotNow, 1, entry);
-   tuple = heap_getnext(scandesc, ForwardScanDirection);
-
-   if (HeapTupleIsValid(tuple))
-       result = HeapTupleGetOid(tuple);
-   else
-       result = InvalidOid;
-
-   heap_endscan(scandesc);
-   heap_close(rel, AccessShareLock);
-
-   return result;
-}
-
-/*
- * get_tablespace_name - given a tablespace OID, look up the name
- *
- * Returns a palloc'd string, or NULL if no such tablespace.
- */
-char *
-get_tablespace_name(Oid spc_oid)
-{
-   char       *result;
-   Relation    rel;
-   HeapScanDesc scandesc;
-   HeapTuple   tuple;
-   ScanKeyData entry[1];
-
-   /* Search pg_tablespace */
-   rel = heap_openr(TableSpaceRelationName, AccessShareLock);
-
-   ScanKeyInit(&entry[0],
-               ObjectIdAttributeNumber,
-               BTEqualStrategyNumber, F_OIDEQ,
-               ObjectIdGetDatum(spc_oid));
-   scandesc = heap_beginscan(rel, SnapshotNow, 1, entry);
-   tuple = heap_getnext(scandesc, ForwardScanDirection);
-
-   /* We assume that there can be at most one matching tuple */
-   if (HeapTupleIsValid(tuple))
-       result = pstrdup(NameStr(((Form_pg_tablespace) GETSTRUCT(tuple))->spcname));
-   else
-       result = NULL;
-
-   heap_endscan(scandesc);
-   heap_close(rel, AccessShareLock);
-
-   return result;
-}
-
 /*
  * Rename a tablespace
  */
@@ -946,6 +872,143 @@ AlterTableSpaceOwner(const char *name, AclId newOwnerSysId)
    heap_close(rel, NoLock);
 }
 
+
+/*
+ * Routines for handling the GUC variable 'default_tablespace'.
+ */
+
+/* assign_hook: validate new default_tablespace, do extra actions as needed */
+const char *
+assign_default_tablespace(const char *newval, bool doit, GucSource source)
+{
+   /*
+    * If we aren't inside a transaction, we cannot do database access so
+    * cannot verify the name.  Must accept the value on faith.
+    */
+   if (IsTransactionState())
+   {
+       if (newval[0] != '\0' &&
+           !OidIsValid(get_tablespace_oid(newval)))
+       {
+           if (source >= PGC_S_INTERACTIVE)
+               ereport(ERROR,
+                       (errcode(ERRCODE_UNDEFINED_OBJECT),
+                        errmsg("tablespace \"%s\" does not exist",
+                               newval)));
+           return NULL;
+       }
+   }
+
+   return newval;
+}
+
+/*
+ * GetDefaultTablespace -- get the OID of the current default tablespace
+ *
+ * May return InvalidOid to indicate "use the database's default tablespace"
+ *
+ * This exists to hide (and possibly optimize the use of) the
+ * default_tablespace GUC variable.
+ */
+Oid
+GetDefaultTablespace(void)
+{
+   Oid         result;
+
+   /* Fast path for default_tablespace == "" */
+   if (default_tablespace == NULL || default_tablespace[0] == '\0')
+       return InvalidOid;
+   /*
+    * It is tempting to cache this lookup for more speed, but then we would
+    * fail to detect the case where the tablespace was dropped since the
+    * GUC variable was set.  Note also that we don't complain if the value
+    * fails to refer to an existing tablespace; we just silently return
+    * InvalidOid, causing the new object to be created in the database's
+    * tablespace.
+    */
+   result = get_tablespace_oid(default_tablespace);
+   /*
+    * Allow explicit specification of database's default tablespace in
+    * default_tablespace without triggering permissions checks.
+    */
+   if (result == MyDatabaseTableSpace)
+       result = InvalidOid;
+   return result;
+}
+
+
+/*
+ * get_tablespace_oid - given a tablespace name, look up the OID
+ *
+ * Returns InvalidOid if tablespace name not found.
+ */
+Oid
+get_tablespace_oid(const char *tablespacename)
+{
+   Oid         result;
+   Relation    rel;
+   HeapScanDesc scandesc;
+   HeapTuple   tuple;
+   ScanKeyData entry[1];
+
+   /* Search pg_tablespace */
+   rel = heap_openr(TableSpaceRelationName, AccessShareLock);
+
+   ScanKeyInit(&entry[0],
+               Anum_pg_tablespace_spcname,
+               BTEqualStrategyNumber, F_NAMEEQ,
+               CStringGetDatum(tablespacename));
+   scandesc = heap_beginscan(rel, SnapshotNow, 1, entry);
+   tuple = heap_getnext(scandesc, ForwardScanDirection);
+
+   if (HeapTupleIsValid(tuple))
+       result = HeapTupleGetOid(tuple);
+   else
+       result = InvalidOid;
+
+   heap_endscan(scandesc);
+   heap_close(rel, AccessShareLock);
+
+   return result;
+}
+
+/*
+ * get_tablespace_name - given a tablespace OID, look up the name
+ *
+ * Returns a palloc'd string, or NULL if no such tablespace.
+ */
+char *
+get_tablespace_name(Oid spc_oid)
+{
+   char       *result;
+   Relation    rel;
+   HeapScanDesc scandesc;
+   HeapTuple   tuple;
+   ScanKeyData entry[1];
+
+   /* Search pg_tablespace */
+   rel = heap_openr(TableSpaceRelationName, AccessShareLock);
+
+   ScanKeyInit(&entry[0],
+               ObjectIdAttributeNumber,
+               BTEqualStrategyNumber, F_OIDEQ,
+               ObjectIdGetDatum(spc_oid));
+   scandesc = heap_beginscan(rel, SnapshotNow, 1, entry);
+   tuple = heap_getnext(scandesc, ForwardScanDirection);
+
+   /* We assume that there can be at most one matching tuple */
+   if (HeapTupleIsValid(tuple))
+       result = pstrdup(NameStr(((Form_pg_tablespace) GETSTRUCT(tuple))->spcname));
+   else
+       result = NULL;
+
+   heap_endscan(scandesc);
+   heap_close(rel, AccessShareLock);
+
+   return result;
+}
+
+
 /*
  * TABLESPACE resource manager's routines
  */
index fe74495781eb38d3f2da263260e2ffcdb2ffef61..093965a51745463d3fe3c4be62ffea2fd7249715 100644 (file)
@@ -15,7 +15,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/nodes/copyfuncs.c,v 1.292 2004/08/29 05:06:43 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/nodes/copyfuncs.c,v 1.293 2004/11/05 19:15:59 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -2401,7 +2401,6 @@ _copyCreateSchemaStmt(CreateSchemaStmt *from)
 
    COPY_STRING_FIELD(schemaname);
    COPY_STRING_FIELD(authid);
-   COPY_STRING_FIELD(tablespacename);
    COPY_NODE_FIELD(schemaElts);
 
    return newnode;
index 22c94cb4af2f41b7c2826f5cb8762f84d1cfbe31..6099f42cca749d3d9a7981c723dc1ecb28a43594 100644 (file)
@@ -18,7 +18,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/nodes/equalfuncs.c,v 1.231 2004/08/29 05:06:43 momjian Exp $
+ *   $PostgreSQL: pgsql/src/backend/nodes/equalfuncs.c,v 1.232 2004/11/05 19:15:59 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1377,7 +1377,6 @@ _equalCreateSchemaStmt(CreateSchemaStmt *a, CreateSchemaStmt *b)
 {
    COMPARE_STRING_FIELD(schemaname);
    COMPARE_STRING_FIELD(authid);
-   COMPARE_STRING_FIELD(tablespacename);
    COMPARE_NODE_FIELD(schemaElts);
 
    return true;
index 5a12bf118919d65a08b4e241b4d7a9e6ec014125..12c1acac4d8d7c583f43030a33b7df358206de52 100644 (file)
@@ -11,7 +11,7 @@
  *
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/parser/gram.y,v 2.478 2004/10/01 16:39:59 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/parser/gram.y,v 2.479 2004/11/05 19:16:02 tgl Exp $
  *
  * HISTORY
  *   AUTHOR            DATE            MAJOR EVENT
@@ -780,7 +780,7 @@ DropGroupStmt:
  *****************************************************************************/
 
 CreateSchemaStmt:
-           CREATE SCHEMA OptSchemaName AUTHORIZATION UserId OptTableSpace OptSchemaEltList
+           CREATE SCHEMA OptSchemaName AUTHORIZATION UserId OptSchemaEltList
                {
                    CreateSchemaStmt *n = makeNode(CreateSchemaStmt);
                    /* One can omit the schema name or the authorization id. */
@@ -789,18 +789,16 @@ CreateSchemaStmt:
                    else
                        n->schemaname = $5;
                    n->authid = $5;
-                   n->tablespacename = $6;
-                   n->schemaElts = $7;
+                   n->schemaElts = $6;
                    $$ = (Node *)n;
                }
-           | CREATE SCHEMA ColId OptTableSpace OptSchemaEltList
+           | CREATE SCHEMA ColId OptSchemaEltList
                {
                    CreateSchemaStmt *n = makeNode(CreateSchemaStmt);
                    /* ...but not both */
                    n->schemaname = $3;
                    n->authid = NULL;
-                   n->tablespacename = $4;
-                   n->schemaElts = $5;
+                   n->schemaElts = $4;
                    $$ = (Node *)n;
                }
        ;
index bb0fc2a1794ac7e6371f6167fdefbb95187144df..36e7208dc1bd08f46d459a65c08a3e78c7452970 100644 (file)
@@ -3,7 +3,7 @@
  *             back to source text
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.184 2004/10/27 18:09:38 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.185 2004/11/05 19:16:11 tgl Exp $
  *
  *   This software is copyrighted by Jan Wieck - Hamburg.
  *
@@ -55,7 +55,6 @@
 #include "catalog/pg_operator.h"
 #include "catalog/pg_shadow.h"
 #include "catalog/pg_trigger.h"
-#include "commands/tablespace.h"
 #include "executor/spi.h"
 #include "lib/stringinfo.h"
 #include "miscadmin.h"
@@ -163,7 +162,6 @@ static char *pg_get_constraintdef_worker(Oid constraintId, bool fullCommand,
                            int prettyFlags);
 static char *pg_get_expr_worker(text *expr, Oid relid, char *relname,
                   int prettyFlags);
-static Oid get_constraint_index(Oid constraintRelOid, Oid constraintOid);
 static void make_ruledef(StringInfo buf, HeapTuple ruletup, TupleDesc rulettc,
             int prettyFlags);
 static void make_viewdef(StringInfo buf, HeapTuple ruletup, TupleDesc rulettc,
@@ -774,27 +772,6 @@ pg_get_indexdef_worker(Oid indexrelid, int colno, int prettyFlags)
    {
        appendStringInfoChar(&buf, ')');
 
-       /*
-        * If the index is in a different tablespace from its parent, tell
-        * about that
-        */
-       if (idxrelrec->reltablespace != get_rel_tablespace(indrelid))
-       {
-           char       *spcname;
-
-           if (OidIsValid(idxrelrec->reltablespace))
-               spcname = get_tablespace_name(idxrelrec->reltablespace);
-           else
-               spcname = get_tablespace_name(MyDatabaseTableSpace);
-
-           if (spcname)        /* just paranoia... */
-           {
-               appendStringInfo(&buf, " TABLESPACE %s",
-                                quote_identifier(spcname));
-               pfree(spcname);
-           }
-       }
-
        /*
         * If it's a partial index, decompile and append the predicate
         */
@@ -1023,7 +1000,6 @@ pg_get_constraintdef_worker(Oid constraintId, bool fullCommand,
            {
                Datum       val;
                bool        isnull;
-               Oid         indexOid;
 
                /* Start off the constraint definition */
                if (conForm->contype == CONSTRAINT_PRIMARY)
@@ -1041,30 +1017,6 @@ pg_get_constraintdef_worker(Oid constraintId, bool fullCommand,
                decompile_column_index_array(val, conForm->conrelid, &buf);
 
                appendStringInfo(&buf, ")");
-
-               /* Add TABLESPACE if it's not default */
-               indexOid = get_constraint_index(RelationGetRelid(conDesc),
-                                               constraintId);
-               if (OidIsValid(indexOid))
-               {
-                   Oid         reltablespace;
-                   Oid         indtablespace;
-
-                   reltablespace = get_rel_tablespace(conForm->conrelid);
-                   indtablespace = get_rel_tablespace(indexOid);
-                   if (OidIsValid(indtablespace) &&
-                       indtablespace != reltablespace)
-                   {
-                       char       *spcname = get_tablespace_name(indtablespace);
-
-                       if (spcname)    /* just paranoia... */
-                       {
-                           appendStringInfo(&buf, " USING INDEX TABLESPACE %s",
-                                            quote_identifier(spcname));
-                           pfree(spcname);
-                       }
-                   }
-               }
                break;
            }
        case CONSTRAINT_CHECK:
@@ -1377,67 +1329,6 @@ pg_get_serial_sequence(PG_FUNCTION_ARGS)
    PG_RETURN_NULL();
 }
 
-/*
- * get_constraint_index
- *     Given the OID of a unique or primary-key constraint,
- *     look up the OID of the underlying index.
- *
- * We make the caller pass in the OID of pg_constraint, too, simply because
- * it's probably got it at hand already.
- *
- * Returns InvalidOid if index can't be found.
- */
-static Oid
-get_constraint_index(Oid constraintRelOid, Oid constraintOid)
-{
-   Oid         result = InvalidOid;
-   Relation    depRel;
-   ScanKeyData key[3];
-   SysScanDesc scan;
-   HeapTuple   tup;
-
-   /* Search the dependency table for the dependent index */
-   depRel = heap_openr(DependRelationName, AccessShareLock);
-
-   ScanKeyInit(&key[0],
-               Anum_pg_depend_refclassid,
-               BTEqualStrategyNumber, F_OIDEQ,
-               ObjectIdGetDatum(constraintRelOid));
-   ScanKeyInit(&key[1],
-               Anum_pg_depend_refobjid,
-               BTEqualStrategyNumber, F_OIDEQ,
-               ObjectIdGetDatum(constraintOid));
-   ScanKeyInit(&key[2],
-               Anum_pg_depend_refobjsubid,
-               BTEqualStrategyNumber, F_INT4EQ,
-               Int32GetDatum(0));
-
-   scan = systable_beginscan(depRel, DependReferenceIndex, true,
-                             SnapshotNow, 3, key);
-
-   while (HeapTupleIsValid(tup = systable_getnext(scan)))
-   {
-       Form_pg_depend deprec = (Form_pg_depend) GETSTRUCT(tup);
-
-       /*
-        * We assume any internal dependency of a relation on the
-        * constraint must be what we are looking for.
-        */
-       if (deprec->classid == RelOid_pg_class &&
-           deprec->objsubid == 0 &&
-           deprec->deptype == DEPENDENCY_INTERNAL)
-       {
-           result = deprec->objid;
-           break;
-       }
-   }
-
-   systable_endscan(scan);
-   heap_close(depRel, AccessShareLock);
-
-   return result;
-}
-
 
 /* ----------
  * deparse_expression          - General utility for deparsing expressions
index 2e3848947c85092b849449f3ae38bffc8a33efa1..3e63efcc4f6f554a3b3773efddebb187bef59871 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/cache/lsyscache.c,v 1.117 2004/10/20 16:04:49 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/cache/lsyscache.c,v 1.118 2004/11/05 19:16:14 tgl Exp $
  *
  * NOTES
  *   Eventually, the index information should go through here, too.
@@ -985,34 +985,6 @@ get_rel_namespace(Oid relid)
        return InvalidOid;
 }
 
-/*
- * get_rel_tablespace
- *     Returns the pg_tablespace OID associated with a given relation.
- *
- * Note: failure return is InvalidOid, which cannot be distinguished from
- * "default tablespace for this database", but that seems OK.
- */
-Oid
-get_rel_tablespace(Oid relid)
-{
-   HeapTuple   tp;
-
-   tp = SearchSysCache(RELOID,
-                       ObjectIdGetDatum(relid),
-                       0, 0, 0);
-   if (HeapTupleIsValid(tp))
-   {
-       Form_pg_class reltup = (Form_pg_class) GETSTRUCT(tp);
-       Oid         result;
-
-       result = reltup->reltablespace;
-       ReleaseSysCache(tp);
-       return result;
-   }
-   else
-       return InvalidOid;
-}
-
 /*
  * get_rel_type_id
  *
@@ -2044,34 +2016,6 @@ get_namespace_name(Oid nspid)
        return NULL;
 }
 
-/*
- * get_namespace_tablespace
- *     Returns the default tablespace of a given namespace
- *
- * Note: failure return is InvalidOid, which cannot be distinguished from
- * "default tablespace for this database", but that seems OK.
- */
-Oid
-get_namespace_tablespace(Oid nspid)
-{
-   HeapTuple   tp;
-
-   tp = SearchSysCache(NAMESPACEOID,
-                       ObjectIdGetDatum(nspid),
-                       0, 0, 0);
-   if (HeapTupleIsValid(tp))
-   {
-       Form_pg_namespace nsptup = (Form_pg_namespace) GETSTRUCT(tp);
-       Oid         result;
-
-       result = nsptup->nsptablespace;
-       ReleaseSysCache(tp);
-       return result;
-   }
-   else
-       return InvalidOid;
-}
-
 /*             ---------- PG_SHADOW CACHE ----------                    */
 
 /*
index c742ace1b5c75d6d151ead69804348b7227599d7..b59644397cf03f300495f7324bf4c84511d0c9bd 100644 (file)
@@ -10,7 +10,7 @@
  * Written by Peter Eisentraut .
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.247 2004/11/04 19:08:42 tgl Exp $
+ *   $PostgreSQL: pgsql/src/backend/utils/misc/guc.c,v 1.248 2004/11/05 19:16:16 tgl Exp $
  *
  *--------------------------------------------------------------------
  */
@@ -78,6 +78,7 @@ extern DLLIMPORT bool check_function_bodies;
 extern int CommitDelay;
 extern int CommitSiblings;
 extern int DebugSharedBuffers;
+extern char *default_tablespace;
 
 static const char *assign_log_destination(const char *value,
                       bool doit, GucSource source);
@@ -1506,6 +1507,15 @@ static struct config_string ConfigureNamesString[] =
        "ISO, MDY", assign_datestyle, NULL
    },
 
+   {
+       {"default_tablespace", PGC_USERSET, CLIENT_CONN_STATEMENT,
+           gettext_noop("Sets the default tablespace to create tables and indexes in."),
+           gettext_noop("An empty string selects the database's default tablespace.")
+       },
+       &default_tablespace,
+       "", assign_default_tablespace, NULL
+   },
+
    {
        {"default_transaction_isolation", PGC_USERSET, CLIENT_CONN_STATEMENT,
            gettext_noop("Sets the transaction isolation level of each new transaction."),
index 3aed76a411ee03c153390345a56d69562370f696..86640efa7da7b2ec818f7b815f808c89d77dcdda 100644 (file)
 # - Statement Behavior -
 
 #search_path = '$user,public'  # schema names
+#default_tablespace = ''   # a tablespace name, or '' for default
 #check_function_bodies = true
 #default_transaction_isolation = 'read committed'
 #default_transaction_read_only = false
index 8b524c85613bb4aa1469d3b1fbecddbe51dc4e3a..45510a98794817087379e31303a99a3ee95b31a5 100644 (file)
@@ -12,7 +12,7 @@
  * by PostgreSQL
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/bin/pg_dump/pg_dump.c,v 1.390 2004/10/22 16:04:35 petere Exp $
+ *   $PostgreSQL: pgsql/src/bin/pg_dump/pg_dump.c,v 1.391 2004/11/05 19:16:19 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -108,9 +108,6 @@ static const CatalogId nilCatalogId = {0, 0};
 static NamespaceInfo *g_namespaces;
 static int g_numNamespaces;
 
-/* need the name of the database's default tablespace */
-static char *dbDefaultTableSpace;
-
 /* flag to turn on/off dollar quoting */
 static int disable_dollar_quoting = 0;
 
@@ -1252,9 +1249,6 @@ dumpDatabase(Archive *AH)
    encoding = PQgetvalue(res, 0, i_encoding);
    tablespace = PQgetvalue(res, 0, i_tablespace);
 
-   /* save dattablespace name for later dump routines */
-   dbDefaultTableSpace = strdup(tablespace);
-
    appendPQExpBuffer(creaQry, "CREATE DATABASE %s WITH TEMPLATE = template0",
                      fmtId(datname));
    if (strlen(encoding) > 0)
@@ -1482,7 +1476,6 @@ getNamespaces(int *numNamespaces)
    int         i_oid;
    int         i_nspname;
    int         i_usename;
-   int         i_nsptablespace;
    int         i_nspacl;
 
    /*
@@ -1500,7 +1493,6 @@ getNamespaces(int *numNamespaces)
        nsinfo[0].dobj.name = strdup("public");
        nsinfo[0].usename = strdup("");
        nsinfo[0].nspacl = strdup("");
-       nsinfo[0].nsptablespace = strdup("");
 
        selectDumpableNamespace(&nsinfo[0]);
 
@@ -1511,7 +1503,6 @@ getNamespaces(int *numNamespaces)
        nsinfo[1].dobj.name = strdup("pg_catalog");
        nsinfo[1].usename = strdup("");
        nsinfo[1].nspacl = strdup("");
-       nsinfo[0].nsptablespace = strdup("");
 
        selectDumpableNamespace(&nsinfo[1]);
 
@@ -1530,21 +1521,9 @@ getNamespaces(int *numNamespaces)
     * we fetch all namespaces including system ones, so that every object
     * we read in can be linked to a containing namespace.
     */
-   if (g_fout->remoteVersion >= 80000)
-   {
-       appendPQExpBuffer(query, "SELECT tableoid, oid, nspname, "
-                         "(select usename from pg_user where nspowner = usesysid) as usename, "
-                         "nspacl, "
-                         "(SELECT spcname FROM pg_tablespace t WHERE t.oid = nsptablespace) AS nsptablespace "
-                         "FROM pg_namespace");
-   }
-   else
-   {
-       appendPQExpBuffer(query, "SELECT tableoid, oid, nspname, "
-                         "(select usename from pg_user where nspowner = usesysid) as usename, "
-                         "nspacl, NULL AS nsptablespace "
-                         "FROM pg_namespace");
-   }
+   appendPQExpBuffer(query, "SELECT tableoid, oid, nspname, "
+                     "(select usename from pg_user where nspowner = usesysid) as usename, "
+                     "nspacl FROM pg_namespace");
 
    res = PQexec(g_conn, query->data);
    check_sql_result(res, g_conn, query->data, PGRES_TUPLES_OK);
@@ -1558,7 +1537,6 @@ getNamespaces(int *numNamespaces)
    i_nspname = PQfnumber(res, "nspname");
    i_usename = PQfnumber(res, "usename");
    i_nspacl = PQfnumber(res, "nspacl");
-   i_nsptablespace = PQfnumber(res, "nsptablespace");
 
    for (i = 0; i < ntups; i++)
    {
@@ -1569,7 +1547,6 @@ getNamespaces(int *numNamespaces)
        nsinfo[i].dobj.name = strdup(PQgetvalue(res, i, i_nspname));
        nsinfo[i].usename = strdup(PQgetvalue(res, i, i_usename));
        nsinfo[i].nspacl = strdup(PQgetvalue(res, i, i_nspacl));
-       nsinfo[i].nsptablespace = strdup(PQgetvalue(res, i, i_nsptablespace));
 
        /* Decide whether to dump this namespace */
        selectDumpableNamespace(&nsinfo[i]);
@@ -4432,16 +4409,9 @@ dumpNamespace(Archive *fout, NamespaceInfo *nspinfo)
     */
    appendPQExpBuffer(delq, "DROP SCHEMA %s;\n", qnspname);
 
-   appendPQExpBuffer(q, "CREATE SCHEMA %s AUTHORIZATION %s",
+   appendPQExpBuffer(q, "CREATE SCHEMA %s AUTHORIZATION %s;\n",
                      qnspname, fmtId(nspinfo->usename));
 
-   /* Add tablespace qualifier, if not default for database */
-   if (strlen(nspinfo->nsptablespace) != 0)
-       appendPQExpBuffer(q, " TABLESPACE %s",
-                         fmtId(nspinfo->nsptablespace));
-
-   appendPQExpBuffer(q, ";\n");
-
    ArchiveEntry(fout, nspinfo->dobj.catId, nspinfo->dobj.dumpId,
                 nspinfo->dobj.name,
                 NULL, strcmp(nspinfo->dobj.name, "public") == 0 ? nspinfo->usename : "",
@@ -6659,17 +6629,10 @@ dumpTableSchema(Archive *fout, TableInfo *tbinfo)
            appendPQExpBuffer(q, ")");
        }
 
-       /* Output tablespace clause if different from parent schema's */
-       if (strcmp(tbinfo->reltablespace,
-                  tbinfo->dobj.namespace->nsptablespace) != 0)
-       {
-           if (strlen(tbinfo->reltablespace) != 0)
-               appendPQExpBuffer(q, " TABLESPACE %s",
-                                 fmtId(tbinfo->reltablespace));
-           else if (strlen(dbDefaultTableSpace) != 0)
-               appendPQExpBuffer(q, " TABLESPACE %s",
-                                 fmtId(dbDefaultTableSpace));
-       }
+       /* Output tablespace clause if not database's default */
+       if (strlen(tbinfo->reltablespace) != 0)
+           appendPQExpBuffer(q, " TABLESPACE %s",
+                             fmtId(tbinfo->reltablespace));
 
        appendPQExpBuffer(q, ";\n");
 
@@ -6957,17 +6920,10 @@ dumpConstraint(Archive *fout, ConstraintInfo *coninfo)
 
        appendPQExpBuffer(q, ")");
 
-       /* Output tablespace clause if different from parent table's */
-       if (strcmp(indxinfo->tablespace,
-                  indxinfo->indextable->reltablespace) != 0)
-       {
-           if (strlen(indxinfo->tablespace) != 0)
-               appendPQExpBuffer(q, " USING INDEX TABLESPACE %s",
-                                 fmtId(indxinfo->tablespace));
-           else if (strlen(dbDefaultTableSpace) != 0)
-               appendPQExpBuffer(q, " USING INDEX TABLESPACE %s",
-                                 fmtId(dbDefaultTableSpace));
-       }
+       /* Output tablespace clause if not database's default */
+       if (strlen(indxinfo->tablespace) != 0)
+           appendPQExpBuffer(q, " USING INDEX TABLESPACE %s",
+                             fmtId(indxinfo->tablespace));
 
        appendPQExpBuffer(q, ";\n");
 
index 6c3c02707afec7319093baa64b753badc2dc72bf..07798c90f4720e3fd155f04c2ec3e798362da990 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2004, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/bin/pg_dump/pg_dump.h,v 1.112 2004/08/29 05:06:53 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/pg_dump/pg_dump.h,v 1.113 2004/11/05 19:16:19 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -98,7 +98,6 @@ typedef struct _namespaceInfo
    DumpableObject dobj;
    char       *usename;        /* name of owner, or empty string */
    char       *nspacl;
-   char       *nsptablespace;  /* default tablespace */
    bool        dump;           /* true if need to dump definition */
 } NamespaceInfo;
 
index 32c7bcbaf6b7c82aa70c473596661f6513e78687..21ae52b82b3c16de09e4c9a14463be596fa0f556 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2004, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/describe.c,v 1.108 2004/10/12 21:54:44 petere Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/describe.c,v 1.109 2004/11/05 19:16:21 tgl Exp $
  */
 #include "postgres_fe.h"
 #include "describe.h"
@@ -112,7 +112,7 @@ describeTablespaces(const char *pattern, bool verbose)
    PGresult   *res;
    printQueryOpt myopt = pset.popt;
 
-   if (pset.sversion < 70500)
+   if (pset.sversion < 80000)
    {
        fprintf(stderr, _("The server version (%d) does not support tablespaces.\n"),
                pset.sversion);
@@ -715,7 +715,7 @@ describeOneTableDetails(const char *schemaname,
    "SELECT relhasindex, relkind, relchecks, reltriggers, relhasrules, \n"
                      "relhasoids %s \n"
                      "FROM pg_catalog.pg_class WHERE oid = '%s'",
-                     pset.sversion >= 70500 ? ", reltablespace" : "",
+                     pset.sversion >= 80000 ? ", reltablespace" : "",
                      oid);
    res = PSQLexec(buf.data, false);
    if (!res)
@@ -737,7 +737,7 @@ describeOneTableDetails(const char *schemaname,
    tableinfo.hasindex = strcmp(PQgetvalue(res, 0, 0), "t") == 0;
    tableinfo.hasrules = strcmp(PQgetvalue(res, 0, 4), "t") == 0;
    tableinfo.hasoids = strcmp(PQgetvalue(res, 0, 5), "t") == 0;
-   tableinfo.tablespace = (pset.sversion >= 70500) ?
+   tableinfo.tablespace = (pset.sversion >= 80000) ?
        atooid(PQgetvalue(res, 0, 6)) : 0;
    PQclear(res);
 
index cd448b9d0f15333ff6c1b79f8efbd4e0ca0f09ac..93f576a0bf41e5c106a1e9ce2eac7af36f3c45e0 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2004, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/tab-complete.c,v 1.117 2004/11/02 16:10:05 petere Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/tab-complete.c,v 1.118 2004/11/05 19:16:22 tgl Exp $
  */
 
 /*----------------------------------------------------------------------
@@ -526,6 +526,7 @@ psql_completion(char *text, int start, int end)
        "debug_print_plan",
        "debug_print_rewritten",
        "default_statistics_target",
+       "default_tablespace",
        "default_transaction_isolation",
        "default_transaction_read_only",
        "default_with_oids",
index 32c097eeb011c17a96c6e17e14180502382a6a55..25e34f81537df8a1627b95cd042f6eca8948a118 100644 (file)
@@ -37,7 +37,7 @@
  * Portions Copyright (c) 1996-2004, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/catalog/catversion.h,v 1.251 2004/10/11 17:24:40 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/catalog/catversion.h,v 1.252 2004/11/05 19:16:32 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -53,6 +53,6 @@
  */
 
 /*                         yyyymmddN */
-#define CATALOG_VERSION_NO 200410111
+#define CATALOG_VERSION_NO 200411041
 
 #endif
index 795663931b105152f13ad17630565b744255c813..33c8a59e2736527bbb6266e2aaba0c8fed447eb4 100644 (file)
@@ -8,7 +8,7 @@
  * Portions Copyright (c) 1996-2004, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/catalog/pg_namespace.h,v 1.14 2004/08/29 05:06:55 momjian Exp $
+ * $PostgreSQL: pgsql/src/include/catalog/pg_namespace.h,v 1.15 2004/11/05 19:16:33 tgl Exp $
  *
  * NOTES
  *   the genbki.sh script reads this file and generates .bki
@@ -40,7 +40,6 @@ CATALOG(pg_namespace)
 {
    NameData    nspname;
    int4        nspowner;
-   Oid         nsptablespace;  /* default table space for name space */
    aclitem     nspacl[1];      /* VARIABLE LENGTH FIELD */
 } FormData_pg_namespace;
 
@@ -56,11 +55,10 @@ typedef FormData_pg_namespace *Form_pg_namespace;
  * ----------------
  */
 
-#define Natts_pg_namespace             4
+#define Natts_pg_namespace             3
 #define Anum_pg_namespace_nspname      1
 #define Anum_pg_namespace_nspowner     2
-#define Anum_pg_namespace_nsptablespace 3
-#define Anum_pg_namespace_nspacl       4
+#define Anum_pg_namespace_nspacl       3
 
 
 /* ----------------
@@ -68,13 +66,13 @@ typedef FormData_pg_namespace *Form_pg_namespace;
  * ---------------
  */
 
-DATA(insert OID = 11 ( "pg_catalog" PGUID _null_ ));
+DATA(insert OID = 11 ( "pg_catalog" PGUID _null_ ));
 DESCR("System catalog schema");
 #define PG_CATALOG_NAMESPACE 11
-DATA(insert OID = 99 ( "pg_toast" PGUID _null_ ));
+DATA(insert OID = 99 ( "pg_toast" PGUID _null_ ));
 DESCR("Reserved schema for TOAST tables");
 #define PG_TOAST_NAMESPACE 99
-DATA(insert OID = 2200 ( "public" PGUID _null_ ));
+DATA(insert OID = 2200 ( "public" PGUID _null_ ));
 DESCR("Standard public schema");
 #define PG_PUBLIC_NAMESPACE 2200
 
@@ -82,7 +80,6 @@ DESCR("Standard public schema");
 /*
  * prototypes for functions in pg_namespace.c
  */
-extern Oid NamespaceCreate(const char *nspName, int32 ownerSysId,
-               Oid nspTablespace);
+extern Oid NamespaceCreate(const char *nspName, int32 ownerSysId);
 
 #endif   /* PG_NAMESPACE_H */
index 58ad23ae161c14cc855339add6710504ff0a2807..a6c21888245c7233f50966bf3923741679b0be58 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2004, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/commands/tablespace.h,v 1.6 2004/10/17 20:47:21 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/commands/tablespace.h,v 1.7 2004/11/05 19:16:36 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -32,6 +32,7 @@ typedef struct xl_tblspc_drop_rec
    Oid         ts_id;
 }  xl_tblspc_drop_rec;
 
+
 extern void CreateTableSpace(CreateTableSpaceStmt *stmt);
 extern void DropTableSpace(DropTableSpaceStmt *stmt);
 extern void RenameTableSpace(const char *oldname, const char *newname);
@@ -39,6 +40,8 @@ extern void AlterTableSpaceOwner(const char *name, AclId newOwnerSysId);
 
 extern void TablespaceCreateDbspace(Oid spcNode, Oid dbNode, bool isRedo);
 
+extern Oid GetDefaultTablespace(void);
+
 extern Oid get_tablespace_oid(const char *tablespacename);
 extern char *get_tablespace_name(Oid spc_oid);
 
index 8b46366bd9784e94efebf06619c93431b1ab591a..9e2dfd839ec3e8503153ddeead14dfac82963692 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2004, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/nodes/parsenodes.h,v 1.269 2004/08/29 05:06:57 momjian Exp $
+ * $PostgreSQL: pgsql/src/include/nodes/parsenodes.h,v 1.270 2004/11/05 19:16:38 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -777,7 +777,6 @@ typedef struct CreateSchemaStmt
    NodeTag     type;
    char       *schemaname;     /* the name of the schema to create */
    char       *authid;         /* the owner of the created schema */
-   char       *tablespacename; /* default tablespace for schema, or NULL */
    List       *schemaElts;     /* schema components (list of parsenodes) */
 } CreateSchemaStmt;
 
index 0559264dd1023acbed9998b5661c0260c44c2f34..19ae296559dba8cdeedf49f19ef30088b00eea9c 100644 (file)
@@ -7,7 +7,7 @@
  * Copyright (c) 2000-2004, PostgreSQL Global Development Group
  * Written by Peter Eisentraut .
  *
- * $PostgreSQL: pgsql/src/include/utils/guc.h,v 1.54 2004/10/22 19:48:19 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/utils/guc.h,v 1.55 2004/11/05 19:16:41 tgl Exp $
  *--------------------------------------------------------------------
  */
 #ifndef GUC_H
@@ -223,6 +223,10 @@ extern void read_nondefault_variables(void);
 /* in utils/adt/datetime.c */
 extern bool ClearDateCache(bool newval, bool doit, GucSource source);
 
+/* in commands/tablespace.c */
+extern const char *assign_default_tablespace(const char *newval,
+                  bool doit, GucSource source);
+
 /* in utils/adt/regexp.c */
 extern const char *assign_regex_flavor(const char *value,
                    bool doit, GucSource source);
index 3a016b7a52d48a40cadbce346aa0a65578a87284..01f6aa73644f355749c7233f713bfea67ffd8a5d 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2004, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $PostgreSQL: pgsql/src/include/utils/lsyscache.h,v 1.91 2004/10/20 16:04:50 tgl Exp $
+ * $PostgreSQL: pgsql/src/include/utils/lsyscache.h,v 1.92 2004/11/05 19:16:41 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -72,7 +72,6 @@ extern Oid    get_relname_relid(const char *relname, Oid relnamespace);
 extern Oid get_system_catalog_relid(const char *catname);
 extern char *get_rel_name(Oid relid);
 extern Oid get_rel_namespace(Oid relid);
-extern Oid get_rel_tablespace(Oid relid);
 extern Oid get_rel_type_id(Oid relid);
 extern char get_rel_relkind(Oid relid);
 extern bool get_typisdefined(Oid typid);
@@ -116,7 +115,6 @@ extern void free_attstatsslot(Oid atttype,
                  Datum *values, int nvalues,
                  float4 *numbers, int nnumbers);
 extern char *get_namespace_name(Oid nspid);
-extern Oid get_namespace_tablespace(Oid nspid);
 extern int32 get_usesysid(const char *username);
 
 #define is_array_type(typid)  (get_element_type(typid) != InvalidOid)
index 0ded0cb6041e116ad2a7763e4ac777d38f79df55..fa579afd6a297ec3fbef1f06889a76d0c2f3364c 100644 (file)
@@ -1,4 +1,4 @@
-/* $PostgreSQL: pgsql/src/interfaces/ecpg/preproc/preproc.y,v 1.299 2004/11/01 13:17:12 davec Exp $ */
+/* $PostgreSQL: pgsql/src/interfaces/ecpg/preproc/preproc.y,v 1.300 2004/11/05 19:16:43 tgl Exp $ */
 
 /* Copyright comment */
 %{
@@ -1013,10 +1013,10 @@ DropGroupStmt: DROP GROUP_P UserId
  *
  *****************************************************************************/
 
-CreateSchemaStmt:  CREATE SCHEMA OptSchemaName AUTHORIZATION UserId OptTableSpace OptSchemaEltList
-           { $$ = cat_str(6, make_str("create schema"), $3, make_str("authorization"), $5, $6, $7); }
-       | CREATE SCHEMA ColId OptTableSpace OptSchemaEltList
-           { $$ = cat_str(4, make_str("create schema"), $3, $4, $5); }
+CreateSchemaStmt:  CREATE SCHEMA OptSchemaName AUTHORIZATION UserId OptSchemaEltList
+           { $$ = cat_str(5, make_str("create schema"), $3, make_str("authorization"), $5, $6); }
+       | CREATE SCHEMA ColId OptSchemaEltList
+           { $$ = cat_str(3, make_str("create schema"), $3, $4); }
        ;
 
 OptSchemaName: ColId       { $$ = $1; }
index 460d4433a844881e4494aecda04149e623da0d64..d094bd70811cabcac08cef4070a17c004b46ec93 100644 (file)
@@ -1,15 +1,11 @@
 -- create a tablespace we can use
 CREATE TABLESPACE testspace LOCATION '@testtablespace@';
 
--- create a schema in the tablespace
-CREATE SCHEMA testschema TABLESPACE testspace;
-
--- sanity check
-SELECT nspname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_namespace n
-   where n.nsptablespace = t.oid and n.nspname = 'testschema';
+-- create a schema we can use
+CREATE SCHEMA testschema;
 
 -- try a table
-CREATE TABLE testschema.foo (i int);
+CREATE TABLE testschema.foo (i int) TABLESPACE testspace;
 SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
     where c.reltablespace = t.oid AND c.relname = 'foo';
 
@@ -17,7 +13,7 @@ INSERT INTO testschema.foo VALUES(1);
 INSERT INTO testschema.foo VALUES(2);
 
 -- index
-CREATE INDEX foo_idx on testschema.foo(i);
+CREATE INDEX foo_idx on testschema.foo(i) TABLESPACE testspace;
 SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
     where c.reltablespace = t.oid AND c.relname = 'foo_idx';
 
index 43044a2bcf86f66e43d3d732b7dbbd6bd003a709..42c4bc628d50f992e190e64bfab31a09661e935f 100644 (file)
@@ -1,17 +1,9 @@
 -- create a tablespace we can use
 CREATE TABLESPACE testspace LOCATION '@testtablespace@';
--- create a schema in the tablespace
-CREATE SCHEMA testschema TABLESPACE testspace;
--- sanity check
-SELECT nspname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_namespace n
-   where n.nsptablespace = t.oid and n.nspname = 'testschema';
-  nspname   |  spcname  
-------------+-----------
- testschema | testspace
-(1 row)
-
+-- create a schema we can use
+CREATE SCHEMA testschema;
 -- try a table
-CREATE TABLE testschema.foo (i int);
+CREATE TABLE testschema.foo (i int) TABLESPACE testspace;
 SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
     where c.reltablespace = t.oid AND c.relname = 'foo';
  relname |  spcname  
@@ -22,7 +14,7 @@ SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
 INSERT INTO testschema.foo VALUES(1);
 INSERT INTO testschema.foo VALUES(2);
 -- index
-CREATE INDEX foo_idx on testschema.foo(i);
+CREATE INDEX foo_idx on testschema.foo(i) TABLESPACE testspace;
 SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
     where c.reltablespace = t.oid AND c.relname = 'foo_idx';
  relname |  spcname