Create a GUC parameter temp_tablespaces that allows selection of the
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 3 Jun 2007 17:08:34 +0000 (17:08 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 3 Jun 2007 17:08:34 +0000 (17:08 +0000)
tablespace(s) in which to store temp tables and temporary files.  This is a
list to allow spreading the load across multiple tablespaces (a random list
element is chosen each time a temp object is to be created).  Temp files are
not stored in per-database pgsql_tmp/ directories anymore, but per-tablespace
directories.

Jaime Casanova and Albert Cervera, with review by Bernd Helmle and Tom Lane.

26 files changed:
doc/src/sgml/config.sgml
doc/src/sgml/manage-ag.sgml
doc/src/sgml/ref/create_index.sgml
doc/src/sgml/ref/create_table.sgml
doc/src/sgml/ref/create_table_as.sgml
doc/src/sgml/ref/drop_tablespace.sgml
doc/src/sgml/ref/grant.sgml
doc/src/sgml/storage.sgml
src/backend/commands/indexcmds.c
src/backend/commands/tablecmds.c
src/backend/commands/tablespace.c
src/backend/executor/execMain.c
src/backend/executor/nodeHash.c
src/backend/executor/nodeHashjoin.c
src/backend/storage/file/buffile.c
src/backend/storage/file/fd.c
src/backend/utils/misc/guc.c
src/backend/utils/misc/postgresql.conf.sample
src/backend/utils/sort/logtape.c
src/backend/utils/sort/tuplestore.c
src/include/commands/tablespace.h
src/include/executor/hashjoin.h
src/include/executor/nodeHashjoin.h
src/include/storage/buffile.h
src/include/storage/fd.h
src/include/utils/guc.h

index f4107d81775d79da7263850dad80a66b0f05d5b7..03fc6741b3f60b747ec04695675317f031ef5921 100644 (file)
@@ -281,7 +281,7 @@ SET ENABLE_SEQSCAN TO OFF;
       If you wish, you can specify the configuration file names and locations
       individually using the parameters <varname>config_file</>,
       <varname>hba_file</> and/or <varname>ident_file</>.
-      <varname>config_file</> can only be specified on the 
+      <varname>config_file</> can only be specified on the
       <command>postgres</command> command line, but the others can be
       set within the main configuration file.  If all three parameters plus
       <varname>data_directory</> are explicitly set, then it is not necessary
@@ -311,7 +311,7 @@ SET ENABLE_SEQSCAN TO OFF;
       <listitem>
        <para>
          Specifies the TCP/IP address(es) on which the server is
-         to listen for connections from client applications.  
+         to listen for connections from client applications.
          The value takes the form of a comma-separated list of host names
          and/or numeric IP addresses.  The special entry <literal>*</>
          corresponds to all available IP interfaces.
@@ -362,7 +362,7 @@ SET ENABLE_SEQSCAN TO OFF;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-superuser-reserved-connections"
      xreflabel="superuser_reserved_connections">
       <term><varname>superuser_reserved_connections</varname>
@@ -461,7 +461,7 @@ SET ENABLE_SEQSCAN TO OFF;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-bonjour-name" xreflabel="bonjour_name">
       <term><varname>bonjour_name</varname> (<type>string</type>)</term>
       <indexterm>
@@ -478,7 +478,7 @@ SET ENABLE_SEQSCAN TO OFF;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-tcp-keepalives-idle" xreflabel="tcp_keepalives_idle">
       <term><varname>tcp_keepalives_idle</varname> (<type>integer</type>)</term>
       <indexterm>
@@ -494,7 +494,7 @@ SET ENABLE_SEQSCAN TO OFF;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-tcp-keepalives-interval" xreflabel="tcp_keepalives_interval">
       <term><varname>tcp_keepalives_interval</varname> (<type>integer</type>)</term>
       <indexterm>
@@ -510,7 +510,7 @@ SET ENABLE_SEQSCAN TO OFF;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-tcp-keepalives-count" xreflabel="tcp_keepalives_count">
       <term><varname>tcp_keepalives_count</varname> (<type>integer</type>)</term>
       <indexterm>
@@ -519,19 +519,19 @@ SET ENABLE_SEQSCAN TO OFF;
       <listitem>
        <para>
         On systems that support the <symbol>TCP_KEEPCNT</symbol> socket option, specifies how
-        many keepalives can be lost before the connection is considered dead. 
+        many keepalives can be lost before the connection is considered dead.
         A value of zero uses the system default. If <symbol>TCP_KEEPCNT</symbol> is not
         supported, this parameter must be zero. This parameter is ignored
         for connections made via a Unix-domain socket.
        </para>
       </listitem>
      </varlistentry>
-     
+
      </variablelist>
      </sect2>
      <sect2 id="runtime-config-connection-security">
      <title>Security and Authentication</title>
-     
+
      <variablelist>
      <varlistentry id="guc-authentication-timeout" xreflabel="authentication_timeout">
       <term><varname>authentication_timeout</varname> (<type>integer</type>)</term>
@@ -826,7 +826,7 @@ SET ENABLE_SEQSCAN TO OFF;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-maintenance-work-mem" xreflabel="maintenance_work_mem">
       <term><varname>maintenance_work_mem</varname> (<type>integer</type>)</term>
       <indexterm>
@@ -896,7 +896,7 @@ SET ENABLE_SEQSCAN TO OFF;
       is not in the map cannot be re-used; instead <productname>PostgreSQL</>
       will request more disk space from the operating system when it needs
       to store new data.
-      The last few lines displayed by a database-wide <command>VACUUM VERBOSE</> 
+      The last few lines displayed by a database-wide <command>VACUUM VERBOSE</>
       command can help in determining if the current settings are adequate.
       A <literal>NOTICE</> message is also printed during such an operation
       if the current settings are too low.
@@ -944,7 +944,7 @@ SET ENABLE_SEQSCAN TO OFF;
        </para>
       </listitem>
      </varlistentry>
-     
+
      </variablelist>
      </sect2>
      <sect2 id="runtime-config-resource-kernel">
@@ -3475,7 +3475,14 @@ SELECT * FROM parent WHERE key = 2400;
         to specify using the default tablespace of the current database.
         If the value does not match the name of any existing tablespace,
         <productname>PostgreSQL</> will automatically use the default
-        tablespace of the current database.
+        tablespace of the current database.  If a nondefault tablespace
+        is specified, the user must have <literal>CREATE</> privilege
+        for it, or creation attempts will fail.
+       </para>
+
+       <para>
+        This variable is not used for temporary tables; for them,
+        <xref linkend="guc-temp-tablespaces"> is consulted instead.
        </para>
 
        <para>
@@ -3485,6 +3492,42 @@ SELECT * FROM parent WHERE key = 2400;
       </listitem>
      </varlistentry>
 
+     <varlistentry id="guc-temp-tablespaces" xreflabel="temp_tablespaces">
+      <term><varname>temp_tablespaces</varname> (<type>string</type>)</term>
+      <indexterm>
+       <primary><varname>temp_tablespaces</> configuration parameter</primary>
+      </indexterm>
+      <indexterm><primary>tablespace</><secondary>temporary</></>
+      <listitem>
+       <para>
+        This variable specifies tablespace(s) in which to create temporary
+        objects (temp tables and indexes on temp tables) when a
+        <command>CREATE</> command does not explicitly specify a tablespace.
+        Temporary files for purposes such as sorting large data sets
+        are also created in these tablespace(s).
+       </para>
+
+       <para>
+        The value is a list of names of tablespaces.  When there is more than
+        one name in the list, <productname>PostgreSQL</> chooses a random
+        member of the list each time a temporary object is to be created.
+       </para>
+
+       <para>
+        If any element of the list is an empty string or does not match the
+        name of any existing tablespace, <productname>PostgreSQL</> will
+        automatically use the default tablespace of the current database
+        instead.  If a nondefault tablespace
+        is specified, the user must have <literal>CREATE</> privilege
+        for it, or creation attempts will fail.
+       </para>
+
+       <para>
+        See also <xref linkend="guc-default-tablespace">.
+       </para>
+      </listitem>
+     </varlistentry>
+
      <varlistentry id="guc-check-function-bodies" xreflabel="check_function_bodies">
       <term><varname>check_function_bodies</varname> (<type>boolean</type>)</term>
       <indexterm>
@@ -3605,7 +3648,7 @@ SELECT * FROM parent WHERE key = 2400;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-xmlbinary" xreflabel="xmlbinary">
       <term><varname>xmlbinary</varname> (<type>string</type>)</term>
       <indexterm>
@@ -3632,7 +3675,7 @@ SELECT * FROM parent WHERE key = 2400;
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-xmloption" xreflabel="xmloption">
       <term><varname>xmloption</varname> (<type>string</type>)</term>
       <indexterm>
@@ -3664,7 +3707,7 @@ SET XML OPTION { DOCUMENT | CONTENT };
        </para>
       </listitem>
      </varlistentry>
-     
+
      </variablelist>
     </sect2>
      <sect2 id="runtime-config-client-format">
@@ -3945,7 +3988,7 @@ dynamic_library_path = 'C:\tools\postgresql;H:\my_project\lib;$libdir'
        </para>
       </listitem>
      </varlistentry>
-     
+
      <varlistentry id="guc-local-preload-libraries" xreflabel="local_preload_libraries">
       <term><varname>local_preload_libraries</varname> (<type>string</type>)</term>
       <indexterm>
@@ -3994,7 +4037,7 @@ dynamic_library_path = 'C:\tools\postgresql;H:\my_project\lib;$libdir'
        <para>
         Every  PostgreSQL-supported library has a <quote>magic
         block</> that is checked to guarantee compatibility.  
-        For this reason, non-PostgreSQL libraries cannot be 
+        For this reason, non-PostgreSQL libraries cannot be
         loaded in this way.
        </para>
       </listitem>
index fa508058663a90d7194828ad2a0996ed81503629..a1d9642dcbed99d9672e207ad913dbf8c046b6e7 100644 (file)
@@ -423,13 +423,23 @@ CREATE TABLE foo(i int);
     do not have an explicit one.
    </para>
 
+   <para>
+    There is also a <xref linkend="guc-temp-tablespaces"> parameter, which
+    determines the placement of temporary tables and indexes, as well as
+    temporary files that are used for purposes such as sorting large data
+    sets.  This can be a list of tablespace names, rather than only one,
+    so that the load associated with temporary objects can be spread over
+    multiple tablespaces.  A random member of the list is picked each time
+    a temporary object is to be created.
+   </para>
+
    <para>
     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 <literal>TABLESPACE</> clause is given (either explicitly or via
-    <varname>default_tablespace</>) when the objects are created.
+    catalogs of that database.  Furthermore, it is the default tablespace
+    used for tables, indexes, and temporary files created within the database,
+    if no <literal>TABLESPACE</> clause is given and no other selection is
+    specified by <varname>default_tablespace</> or
+    <varname>temp_tablespaces</> (as appropriate).
     If a database is created without specifying a tablespace for it,
     it uses the same tablespace as the template database it is copied from.
    </para>
@@ -468,7 +478,7 @@ SELECT spcname FROM pg_tablespace;
    </para>
 
    <para>
-    <productname>PostgreSQL</> makes extensive use of symbolic links
+    <productname>PostgreSQL</> makes use of symbolic links
     to simplify the implementation of tablespaces. This
     means that tablespaces can be used <emphasis>only</> on systems
     that support symbolic links.
index d0b6e60b66f134982e67e574d2bb5f55fbe49e18..23eec80c1ec8e14f7a1308b8f7af665d74ece932 100644 (file)
@@ -240,9 +240,9 @@ CREATE [ UNIQUE ] INDEX [ CONCURRENTLY ] <replaceable class="parameter">name</re
       <listitem>
        <para>
         The tablespace in which to create the index.  If not specified,
-        <xref linkend="guc-default-tablespace"> is used, or the database's
-        default tablespace if <varname>default_tablespace</> is an empty
-        string.
+        <xref linkend="guc-default-tablespace"> is consulted, or
+        <xref linkend="guc-temp-tablespaces"> for indexes on temporary
+        tables.
        </para>
       </listitem>
      </varlistentry>
index b43b3da6bed80654b68139a9043bfc2c90f45d23..0c0573a711232b96c49cf5651277973000df5114 100644 (file)
@@ -645,9 +645,8 @@ and <replaceable class="PARAMETER">table_constraint</replaceable> is:
       The <replaceable class="PARAMETER">tablespace</replaceable> is the name
       of the tablespace in which the new table is to be created.
       If not specified,
-      <xref linkend="guc-default-tablespace"> is used, or the database's
-      default tablespace if <varname>default_tablespace</> is an empty
-      string.
+      <xref linkend="guc-default-tablespace"> is consulted, or
+      <xref linkend="guc-temp-tablespaces"> if the table is temporary.
      </para>
     </listitem>
    </varlistentry>
@@ -660,9 +659,8 @@ and <replaceable class="PARAMETER">table_constraint</replaceable> is:
       associated with a <literal>UNIQUE</literal> or <literal>PRIMARY
       KEY</literal> constraint will be created.
       If not specified,
-      <xref linkend="guc-default-tablespace"> is used, or the database's
-      default tablespace if <varname>default_tablespace</> is an empty
-      string.
+      <xref linkend="guc-default-tablespace"> is consulted, or
+      <xref linkend="guc-temp-tablespaces"> if the table is temporary.
      </para>
     </listitem>
    </varlistentry>
index 5e71533416ad62a7971cb6514f965f582210373f..d26f947fb7f976c225b8f99d9de3bff4bbdcfd48 100644 (file)
@@ -184,9 +184,8 @@ CREATE [ [ GLOBAL | LOCAL ] { TEMPORARY | TEMP } ] TABLE <replaceable>table_name
       The <replaceable class="PARAMETER">tablespace</replaceable> is the name
       of the tablespace in which the new table is to be created.
       If not specified,
-      <xref linkend="guc-default-tablespace"> is used, or the database's
-      default tablespace if <varname>default_tablespace</> is an empty
-      string.
+      <xref linkend="guc-default-tablespace"> is consulted, or
+      <xref linkend="guc-temp-tablespaces"> if the table is temporary.
      </para>
     </listitem>
    </varlistentry>
index 8af5c1ef10415468c14b613c0dfd3d60f7b1772d..9aef801668d81c82f7ec4813a49279e020e0c3b9 100644 (file)
@@ -36,7 +36,10 @@ DROP TABLESPACE [ IF EXISTS ] <replaceable class="PARAMETER">tablespacename</rep
    The tablespace must be empty of all database objects before it can be
    dropped. It is possible that objects in other databases might still reside
    in the tablespace even if no objects in the current database are using
-   the tablespace.
+   the tablespace.  Also, if the tablespace is listed in the <xref
+   linkend="guc-temp-tablespaces"> setting of any active session, the
+   <command>DROP</> might fail due to temporary files residing in the
+   tablespace.
   </para>
  </refsect1>
 
index ccdde1fff3675857bcc6026f6de38ea576eec2a3..e603ed5a1dc0a3ec8bec810498992ea2931870ae 100644 (file)
@@ -211,10 +211,10 @@ GRANT <replaceable class="PARAMETER">role</replaceable> [, ...] TO <replaceable
        have this privilege for the containing schema.
       </para>
       <para>
-       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.)
+       For tablespaces, allows tables, indexes, and temporary files 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.)
       </para>
      </listitem>
     </varlistentry>
index 9831ec14ff2c47993025eacc5ddd55a9aa7101b7..839327add1cf4c6ff61a214ca66f2a80905ab540 100644 (file)
@@ -170,6 +170,17 @@ tablespace is not accessed through <filename>pg_tblspc</>, but corresponds to
 <varname>PGDATA</><filename>/global</>.
 </para>
 
+<para>
+Temporary files (for operations such as sorting more data than can fit in
+memory) are created within <varname>PGDATA</><filename>/base/pgsql_tmp</>,
+or within a <filename>pgsql_tmp</> subdirectory of a tablespace directory
+if a tablespace other than <literal>pg_default</> is specified for them.
+The name of a temporary file has the form
+<filename>pgsql_tmp<replaceable>PPP</>.<replaceable>NNN</></filename>,
+where <replaceable>PPP</> is the PID of the owning backend and
+<replaceable>NNN</> distinguishes different files of that backend.
+</para>
+
 </sect1>
 
 <sect1 id="storage-toast">
index 49cd0f30b33be21adf1a19569a50d502658563a5..d2b1ee4554cbf0db51ac9680aee72f38b0fbcdc4 100644 (file)
@@ -194,7 +194,7 @@ DefineIndex(RangeVar *heapRelation,
        }
 
        /*
-        * Select tablespace to use.  If not specified, use default_tablespace
+        * Select tablespace to use.  If not specified, use default tablespace
         * (which may in turn default to database's default).
         */
        if (tableSpaceName)
@@ -208,7 +208,7 @@ DefineIndex(RangeVar *heapRelation,
        }
        else
        {
-               tablespaceId = GetDefaultTablespace();
+               tablespaceId = GetDefaultTablespace(rel->rd_istemp);
                /* note InvalidOid is OK in this case */
        }
 
index db77baba9feacf02eadd1c1ad6023d38bb4e7dd1..8c4ebf661c99f9a4bb3dae405255972710d2ad4a 100644 (file)
@@ -319,7 +319,7 @@ DefineRelation(CreateStmt *stmt, char relkind)
        }
 
        /*
-        * Select tablespace to use.  If not specified, use default_tablespace
+        * Select tablespace to use.  If not specified, use default tablespace
         * (which may in turn default to database's default).
         */
        if (stmt->tablespacename)
@@ -333,17 +333,10 @@ DefineRelation(CreateStmt *stmt, char relkind)
        }
        else
        {
-               tablespaceId = GetDefaultTablespace();
+               tablespaceId = GetDefaultTablespace(stmt->relation->istemp);
                /* note InvalidOid is OK in this case */
        }
 
-       /*
-        * Parse and validate reloptions, if any.
-        */
-       reloptions = transformRelOptions((Datum) 0, stmt->options, true, false);
-
-       (void) heap_reloptions(relkind, reloptions, true);
-
        /* Check permissions except when using database's default */
        if (OidIsValid(tablespaceId))
        {
@@ -356,6 +349,13 @@ DefineRelation(CreateStmt *stmt, char relkind)
                                                   get_tablespace_name(tablespaceId));
        }
 
+       /*
+        * Parse and validate reloptions, if any.
+        */
+       reloptions = transformRelOptions((Datum) 0, stmt->options, true, false);
+
+       (void) heap_reloptions(relkind, reloptions, true);
+
        /*
         * Look up inheritance ancestors and generate relation schema, including
         * inherited attributes.
index 80dc51d55be7b9bcb943bbae1fb58b139df0667f..8f77c85279255ea878e6c4d9b2fca845ebe21f8f 100644 (file)
 #include "utils/lsyscache.h"
 
 
-/* GUC variable */
+/* GUC variables */
 char      *default_tablespace = NULL;
+char      *temp_tablespaces = NULL;
 
 
 static bool remove_tablespace_directories(Oid tablespaceoid, bool redo);
 static void set_short_version(const char *path);
+static Oid     getTempTablespace(void);
 
 
 /*
@@ -903,16 +905,26 @@ assign_default_tablespace(const char *newval, bool doit, GucSource source)
 /*
  * GetDefaultTablespace -- get the OID of the current default tablespace
  *
- * May return InvalidOid to indicate "use the database's default tablespace"
+ * Regular objects and temporary objects have different default tablespaces,
+ * hence the forTemp parameter must be specified.
+ *
+ * May return InvalidOid to indicate "use the database's default tablespace".
+ *
+ * Note that caller is expected to check appropriate permissions for any
+ * result other than InvalidOid.
  *
  * This exists to hide (and possibly optimize the use of) the
  * default_tablespace GUC variable.
  */
 Oid
-GetDefaultTablespace(void)
+GetDefaultTablespace(bool forTemp)
 {
        Oid                     result;
 
+       /* The temp-table case is handled by getTempTablespace() */
+       if (forTemp)
+               return getTempTablespace();
+
        /* Fast path for default_tablespace == "" */
        if (default_tablespace == NULL || default_tablespace[0] == '\0')
                return InvalidOid;
@@ -936,6 +948,179 @@ GetDefaultTablespace(void)
 }
 
 
+/*
+ * Routines for handling the GUC variable 'temp_tablespaces'.
+ */
+
+/* assign_hook: validate new temp_tablespaces, do extra actions as needed */
+const char *
+assign_temp_tablespaces(const char *newval, bool doit, GucSource source)
+{
+       char       *rawname;
+       List       *namelist;
+       ListCell   *l;
+
+       /* Need a modifiable copy of string */
+       rawname = pstrdup(newval);
+
+       /* Parse string into list of identifiers */
+       if (!SplitIdentifierString(rawname, ',', &namelist))
+       {
+               /* syntax error in name list */
+               pfree(rawname);
+               list_free(namelist);
+               return NULL;
+       }
+
+       /*
+        * If we aren't inside a transaction, we cannot do database access so
+        * cannot verify the individual names.  Must accept the list on faith.
+        */
+       if (source >= PGC_S_INTERACTIVE && IsTransactionState())
+       {
+               foreach(l, namelist)
+               {
+                       char       *curname = (char *) lfirst(l);
+
+                       /* Allow an empty string (signifying database default) */
+                       if (curname[0] == '\0')
+                               continue;
+
+                       /* Else verify that name is a valid tablespace name */
+                       if (get_tablespace_oid(curname) == InvalidOid)
+                               ereport(ERROR,
+                                               (errcode(ERRCODE_UNDEFINED_OBJECT),
+                                                errmsg("tablespace \"%s\" does not exist",
+                                                               curname)));
+               }
+       }
+
+       pfree(rawname);
+       list_free(namelist);
+
+       return newval;
+}
+
+/*
+ * GetTempTablespace -- get the OID of the next temp tablespace to use
+ *
+ * May return InvalidOid to indicate "use the database's default tablespace".
+ *
+ * This is different from GetDefaultTablespace(true) in just two ways:
+ * 1. We check privileges here instead of leaving it to the caller.
+ * 2. It's safe to call this outside a transaction (we just return InvalidOid).
+ * The transaction state check is used so that this can be called from
+ * low-level places that might conceivably run outside a transaction.
+ */
+Oid
+GetTempTablespace(void)
+{
+       Oid                     result;
+
+       /* Can't do catalog access unless within a transaction */
+       if (!IsTransactionState())
+               return InvalidOid;
+
+       /* OK, select a temp tablespace */
+       result = getTempTablespace();
+
+       /* Check permissions except when using database's default */
+       if (OidIsValid(result))
+       {
+               AclResult       aclresult;
+
+               aclresult = pg_tablespace_aclcheck(result, GetUserId(),
+                                                                                  ACL_CREATE);
+               if (aclresult != ACLCHECK_OK)
+                       aclcheck_error(aclresult, ACL_KIND_TABLESPACE,
+                                                  get_tablespace_name(result));
+       }
+
+       return result;
+}
+
+/*
+ * getTempTablespace -- get the OID of the next temp tablespace to use
+ *
+ * This has exactly the API defined for GetDefaultTablespace(true),
+ * in particular that caller is responsible for permissions checks.
+ *
+ * This exists to hide (and possibly optimize the use of) the
+ * temp_tablespaces GUC variable.
+ */
+static Oid
+getTempTablespace(void)
+{
+       Oid                     result;
+       char       *rawname;
+       List       *namelist;
+       int                     nnames;
+       char       *curname;
+
+       if (temp_tablespaces == NULL)
+               return InvalidOid;
+
+       /*
+        * We re-parse the string on each call; this is a bit expensive, but
+        * we don't expect this function will be called many times per query,
+        * so it's probably not worth being tenser.
+        */
+
+       /* Need a modifiable copy of string */
+       rawname = pstrdup(temp_tablespaces);
+
+       /* Parse string into list of identifiers */
+       if (!SplitIdentifierString(rawname, ',', &namelist))
+       {
+               /* syntax error in name list */
+               pfree(rawname);
+               list_free(namelist);
+               return InvalidOid;
+       }
+       nnames = list_length(namelist);
+
+       /* Fast path for temp_tablespaces == "" */
+       if (nnames == 0)
+       {
+               pfree(rawname);
+               list_free(namelist);
+               return InvalidOid;
+       }
+
+       /* Select a random element */
+       if (nnames == 1)                        /* no need for a random() call */
+               curname = (char *) linitial(namelist);
+       else
+               curname = (char *) list_nth(namelist, random() % nnames);
+
+       /*
+        * Empty string means "database's default", else look up the tablespace.
+        *
+        * 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.
+        */
+       if (curname[0] == '\0')
+               result = InvalidOid;
+       else
+               result = get_tablespace_oid(curname);
+
+       /*
+        * Allow explicit specification of database's default tablespace in
+        * temp_tablespaces without triggering permissions checks.
+        */
+       if (result == MyDatabaseTableSpace)
+               result = InvalidOid;
+
+       pfree(rawname);
+       list_free(namelist);
+
+       return result;
+}
+
+
 /*
  * get_tablespace_oid - given a tablespace name, look up the OID
  *
@@ -950,7 +1135,11 @@ get_tablespace_oid(const char *tablespacename)
        HeapTuple       tuple;
        ScanKeyData entry[1];
 
-       /* Search pg_tablespace */
+       /*
+        * Search pg_tablespace.  We use a heapscan here even though there is an
+        * index on name, on the theory that pg_tablespace will usually have just
+        * a few entries and so an indexed lookup is a waste of effort.
+        */
        rel = heap_open(TableSpaceRelationId, AccessShareLock);
 
        ScanKeyInit(&entry[0],
@@ -960,6 +1149,7 @@ get_tablespace_oid(const char *tablespacename)
        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 = HeapTupleGetOid(tuple);
        else
@@ -985,7 +1175,11 @@ get_tablespace_name(Oid spc_oid)
        HeapTuple       tuple;
        ScanKeyData entry[1];
 
-       /* Search pg_tablespace */
+       /*
+        * Search pg_tablespace.  We use a heapscan here even though there is an
+        * index on oid, on the theory that pg_tablespace will usually have just
+        * a few entries and so an indexed lookup is a waste of effort.
+        */
        rel = heap_open(TableSpaceRelationId, AccessShareLock);
 
        ScanKeyInit(&entry[0],
index 0e2b2c7c3723c4c0d7d045dcf151895005509d1f..e35b5043a4fbca001609162528c54bd6c377b4d0 100644 (file)
@@ -2430,7 +2430,7 @@ OpenIntoRel(QueryDesc *queryDesc)
                                           get_namespace_name(namespaceId));
 
        /*
-        * Select tablespace to use.  If not specified, use default_tablespace
+        * Select tablespace to use.  If not specified, use default tablespace
         * (which may in turn default to database's default).
         */
        if (into->tableSpaceName)
@@ -2444,7 +2444,7 @@ OpenIntoRel(QueryDesc *queryDesc)
        }
        else
        {
-               tablespaceId = GetDefaultTablespace();
+               tablespaceId = GetDefaultTablespace(into->rel->istemp);
                /* note InvalidOid is OK in this case */
        }
 
index bf3652451fd540041e15a25c88c06c8adcf377ae..94b3a824a02ef53bdd39c307a2c7c0fb64223714 100644 (file)
@@ -24,6 +24,7 @@
 #include <math.h>
 #include <limits.h>
 
+#include "commands/tablespace.h"
 #include "executor/execdebug.h"
 #include "executor/hashjoin.h"
 #include "executor/instrument.h"
@@ -266,6 +267,7 @@ ExecHashTableCreate(Hash *node, List *hashOperators)
        hashtable->totalTuples = 0;
        hashtable->innerBatchFile = NULL;
        hashtable->outerBatchFile = NULL;
+       hashtable->hashTblSpc = InvalidOid;
        hashtable->spaceUsed = 0;
        hashtable->spaceAllowed = work_mem * 1024L;
 
@@ -325,6 +327,8 @@ ExecHashTableCreate(Hash *node, List *hashOperators)
                hashtable->outerBatchFile = (BufFile **)
                        palloc0(nbatch * sizeof(BufFile *));
                /* The files will not be opened until needed... */
+               /* ... but we want to choose the tablespace only once */
+               hashtable->hashTblSpc = GetTempTablespace();
        }
 
        /*
@@ -506,6 +510,8 @@ ExecHashIncreaseNumBatches(HashJoinTable hashtable)
                        palloc0(nbatch * sizeof(BufFile *));
                hashtable->outerBatchFile = (BufFile **)
                        palloc0(nbatch * sizeof(BufFile *));
+               /* time to choose the tablespace, too */
+               hashtable->hashTblSpc = GetTempTablespace();
        }
        else
        {
@@ -558,7 +564,8 @@ ExecHashIncreaseNumBatches(HashJoinTable hashtable)
                        {
                                /* dump it out */
                                Assert(batchno > curbatch);
-                               ExecHashJoinSaveTuple(HJTUPLE_MINTUPLE(tuple),
+                               ExecHashJoinSaveTuple(hashtable,
+                                                                         HJTUPLE_MINTUPLE(tuple),
                                                                          tuple->hashvalue,
                                                                          &hashtable->innerBatchFile[batchno]);
                                /* and remove from hash table */
@@ -650,7 +657,8 @@ ExecHashTableInsert(HashJoinTable hashtable,
                 * put the tuple into a temp file for later batches
                 */
                Assert(batchno > hashtable->curbatch);
-               ExecHashJoinSaveTuple(tuple,
+               ExecHashJoinSaveTuple(hashtable,
+                                                         tuple,
                                                          hashvalue,
                                                          &hashtable->innerBatchFile[batchno]);
        }
index 46a88fc05200be15a8790f848118154edc83dc98..48d4a43bff395f97f17b8349d3477225257f6564 100644 (file)
@@ -223,7 +223,8 @@ ExecHashJoin(HashJoinState *node)
                                 * in the corresponding outer-batch file.
                                 */
                                Assert(batchno > hashtable->curbatch);
-                               ExecHashJoinSaveTuple(ExecFetchSlotMinimalTuple(outerTupleSlot),
+                               ExecHashJoinSaveTuple(hashtable,
+                                                                         ExecFetchSlotMinimalTuple(outerTupleSlot),
                                                                          hashvalue,
                                                                          &hashtable->outerBatchFile[batchno]);
                                node->hj_NeedNewOuter = true;
@@ -754,7 +755,8 @@ start_over:
  * will get messed up.
  */
 void
-ExecHashJoinSaveTuple(MinimalTuple tuple, uint32 hashvalue,
+ExecHashJoinSaveTuple(HashJoinTable hashtable,
+                                         MinimalTuple tuple, uint32 hashvalue,
                                          BufFile **fileptr)
 {
        BufFile    *file = *fileptr;
@@ -763,7 +765,7 @@ ExecHashJoinSaveTuple(MinimalTuple tuple, uint32 hashvalue,
        if (file == NULL)
        {
                /* First write to this batch file, so open it. */
-               file = BufFileCreateTemp(false);
+               file = BufFileCreateTemp(false, hashtable->hashTblSpc);
                *fileptr = file;
        }
 
index 6bd4ebe491c9c555d9d336cf06dbdcbf07db3734..9a772577c4895a247d2d9a8beb967f7023a51630 100644 (file)
@@ -60,6 +60,7 @@ struct BufFile
         * offsets[i] is the current seek position of files[i].  We use this to
         * avoid making redundant FileSeek calls.
         */
+       Oid                     tblspcOid;              /* tablespace to use (InvalidOid = default) */
 
        bool            isTemp;                 /* can only add files if this is TRUE */
        bool            isInterXact;    /* keep open over transactions? */
@@ -85,7 +86,7 @@ static int    BufFileFlush(BufFile *file);
 
 /*
  * Create a BufFile given the first underlying physical file.
- * NOTE: caller must set isTemp true if appropriate.
+ * NOTE: caller must set tblspcOid, isTemp, isInterXact if appropriate.
  */
 static BufFile *
 makeBufFile(File firstfile)
@@ -97,7 +98,9 @@ makeBufFile(File firstfile)
        file->files[0] = firstfile;
        file->offsets = (long *) palloc(sizeof(long));
        file->offsets[0] = 0L;
+       file->tblspcOid = InvalidOid;
        file->isTemp = false;
+       file->isInterXact = false;
        file->dirty = false;
        file->curFile = 0;
        file->curOffset = 0L;
@@ -116,7 +119,7 @@ extendBufFile(BufFile *file)
        File            pfile;
 
        Assert(file->isTemp);
-       pfile = OpenTemporaryFile(file->isInterXact);
+       pfile = OpenTemporaryFile(file->isInterXact, file->tblspcOid);
        Assert(pfile >= 0);
 
        file->files = (File *) repalloc(file->files,
@@ -133,19 +136,24 @@ extendBufFile(BufFile *file)
  * multiple temporary files if more than MAX_PHYSICAL_FILESIZE bytes are
  * written to it).
  *
+ * If interXact is true, the temp file will not be automatically deleted
+ * at end of transaction.  If tblspcOid is not InvalidOid, the temp file
+ * is created in the specified tablespace instead of the default one.
+ *
  * Note: if interXact is true, the caller had better be calling us in a
  * memory context that will survive across transaction boundaries.
  */
 BufFile *
-BufFileCreateTemp(bool interXact)
+BufFileCreateTemp(bool interXact, Oid tblspcOid)
 {
        BufFile    *file;
        File            pfile;
 
-       pfile = OpenTemporaryFile(interXact);
+       pfile = OpenTemporaryFile(interXact, tblspcOid);
        Assert(pfile >= 0);
 
        file = makeBufFile(pfile);
+       file->tblspcOid = tblspcOid;
        file->isTemp = true;
        file->isInterXact = interXact;
 
index acdcc27901d9b94e6fd59a7a87ede140b01cc6b9..22019db1cef3f5e4d9b2376aca9a4e048b2770bb 100644 (file)
@@ -48,6 +48,7 @@
 
 #include "miscadmin.h"
 #include "access/xact.h"
+#include "catalog/pg_tablespace.h"
 #include "storage/fd.h"
 #include "storage/ipc.h"
 #include "utils/guc.h"
@@ -225,7 +226,7 @@ static File AllocateVfd(void);
 static void FreeVfd(File file);
 
 static int     FileAccess(File file);
-static char *make_database_relative(const char *filename);
+static File OpenTemporaryFileInTablespace(Oid tblspcOid, bool rejectError);
 static void AtProcExit_Files(int code, Datum arg);
 static void CleanupTempFiles(bool isProcExit);
 static void RemovePgTempFilesInDir(const char *tmpdirname);
@@ -721,23 +722,6 @@ FreeVfd(File file)
        VfdCache[0].nextFree = file;
 }
 
-/*
- * make_database_relative()
- *             Prepend DatabasePath to the given file name.
- *
- * Result is a palloc'd string.
- */
-static char *
-make_database_relative(const char *filename)
-{
-       char       *buf;
-
-       Assert(!is_absolute_path(filename));
-       buf = (char *) palloc(strlen(DatabasePath) + strlen(filename) + 2);
-       sprintf(buf, "%s/%s", DatabasePath, filename);
-       return buf;
-}
-
 /* returns 0 on success, -1 on re-open failure (with errno set) */
 static int
 FileAccess(File file)
@@ -844,24 +828,6 @@ PathNameOpenFile(FileName fileName, int fileFlags, int fileMode)
        return file;
 }
 
-/*
- * open a file in the database directory ($PGDATA/base/DIROID/)
- *
- * The passed name MUST be a relative path.  Effectively, this
- * prepends DatabasePath to it and then acts like PathNameOpenFile.
- */
-File
-FileNameOpenFile(FileName fileName, int fileFlags, int fileMode)
-{
-       File            fd;
-       char       *fname;
-
-       fname = make_database_relative(fileName);
-       fd = PathNameOpenFile(fname, fileFlags, fileMode);
-       pfree(fname);
-       return fd;
-}
-
 /*
  * Open a temporary file that will disappear when we close it.
  *
@@ -874,62 +840,110 @@ FileNameOpenFile(FileName fileName, int fileFlags, int fileMode)
  * that created them, so this should be false -- but if you need
  * "somewhat" temporary storage, this might be useful. In either case,
  * the file is removed when the File is explicitly closed.
+ *
+ * tblspcOid: the Oid of the tablespace where the temp file should be created.
+ * If InvalidOid, or if the tablespace can't be found, we silently fall back
+ * to the database's default tablespace.
  */
 File
-OpenTemporaryFile(bool interXact)
+OpenTemporaryFile(bool interXact, Oid tblspcOid)
 {
+       File            file = 0;
+
+       /*
+        * If caller specified a tablespace, try to create there.
+        */
+       if (OidIsValid(tblspcOid))
+               file = OpenTemporaryFileInTablespace(tblspcOid, false);
+
+       /*
+        * If not, or if tablespace is bad, create in database's default
+        * tablespace.  MyDatabaseTableSpace should normally be set before we get
+        * here, but just in case it isn't, fall back to pg_default tablespace.
+        */
+       if (file <= 0)
+               file = OpenTemporaryFileInTablespace(MyDatabaseTableSpace ?
+                                                                                        MyDatabaseTableSpace :
+                                                                                        DEFAULTTABLESPACE_OID,
+                                                                                        true);
+
+       /* Mark it for deletion at close */
+       VfdCache[file].fdstate |= FD_TEMPORARY;
+
+       /* Mark it for deletion at EOXact */
+       if (!interXact)
+       {
+               VfdCache[file].fdstate |= FD_XACT_TEMPORARY;
+               VfdCache[file].create_subid = GetCurrentSubTransactionId();
+       }
+
+       return file;
+}
+
+/*
+ * Open a temporary file in a specific tablespace.
+ * Subroutine for OpenTemporaryFile, which see for details.
+ */
+static File
+OpenTemporaryFileInTablespace(Oid tblspcOid, bool rejectError)
+{
+       char            tempdirpath[MAXPGPATH];
        char            tempfilepath[MAXPGPATH];
        File            file;
 
+       /*
+        * Identify the tempfile directory for this tablespace.
+        *
+        * If someone tries to specify pg_global, use pg_default instead.
+        */
+       if (tblspcOid == DEFAULTTABLESPACE_OID ||
+               tblspcOid == GLOBALTABLESPACE_OID)
+       {
+               /* The default tablespace is {datadir}/base */
+               snprintf(tempdirpath, sizeof(tempdirpath), "base/%s",
+                                PG_TEMP_FILES_DIR);
+       }
+       else
+       {
+               /* All other tablespaces are accessed via symlinks */
+               snprintf(tempdirpath, sizeof(tempdirpath), "pg_tblspc/%u/%s",
+                                tblspcOid, PG_TEMP_FILES_DIR);
+       }
+
        /*
         * Generate a tempfile name that should be unique within the current
         * database instance.
         */
-       snprintf(tempfilepath, sizeof(tempfilepath),
-                        "%s/%s%d.%ld", PG_TEMP_FILES_DIR, PG_TEMP_FILE_PREFIX,
-                        MyProcPid, tempFileCounter++);
+       snprintf(tempfilepath, sizeof(tempfilepath), "%s/%s%d.%ld",
+                        tempdirpath, PG_TEMP_FILE_PREFIX, MyProcPid, tempFileCounter++);
 
        /*
         * Open the file.  Note: we don't use O_EXCL, in case there is an orphaned
         * temp file that can be reused.
         */
-       file = FileNameOpenFile(tempfilepath,
+       file = PathNameOpenFile(tempfilepath,
                                                        O_RDWR | O_CREAT | O_TRUNC | PG_BINARY,
                                                        0600);
        if (file <= 0)
        {
-               char       *dirpath;
-
                /*
-                * We might need to create the pg_tempfiles subdirectory, if no one
-                * has yet done so.
+                * We might need to create the tablespace's tempfile directory,
+                * if no one has yet done so.
                 *
                 * Don't check for error from mkdir; it could fail if someone else
                 * just did the same thing.  If it doesn't work then we'll bomb out on
                 * the second create attempt, instead.
                 */
-               dirpath = make_database_relative(PG_TEMP_FILES_DIR);
-               mkdir(dirpath, S_IRWXU);
-               pfree(dirpath);
+               mkdir(tempdirpath, S_IRWXU);
 
-               file = FileNameOpenFile(tempfilepath,
+               file = PathNameOpenFile(tempfilepath,
                                                                O_RDWR | O_CREAT | O_TRUNC | PG_BINARY,
                                                                0600);
-               if (file <= 0)
+               if (file <= 0 && rejectError)
                        elog(ERROR, "could not create temporary file \"%s\": %m",
                                 tempfilepath);
        }
 
-       /* Mark it for deletion at close */
-       VfdCache[file].fdstate |= FD_TEMPORARY;
-
-       /* Mark it for deletion at EOXact */
-       if (!interXact)
-       {
-               VfdCache[file].fdstate |= FD_XACT_TEMPORARY;
-               VfdCache[file].create_subid = GetCurrentSubTransactionId();
-       }
-
        return file;
 }
 
@@ -1643,27 +1657,32 @@ void
 RemovePgTempFiles(void)
 {
        char            temp_path[MAXPGPATH];
-       DIR                *db_dir;
-       struct dirent *db_de;
+       DIR                *spc_dir;
+       struct dirent *spc_de;
+
+       /*
+        * First process temp files in pg_default ($PGDATA/base)
+        */
+       snprintf(temp_path, sizeof(temp_path), "base/%s", PG_TEMP_FILES_DIR);
+       RemovePgTempFilesInDir(temp_path);
 
        /*
-        * Cycle through pgsql_tmp directories for all databases and remove old
-        * temp files.
+        * Cycle through temp directories for all non-default tablespaces.
         */
-       db_dir = AllocateDir("base");
+       spc_dir = AllocateDir("pg_tblspc");
 
-       while ((db_de = ReadDir(db_dir, "base")) != NULL)
+       while ((spc_de = ReadDir(spc_dir, "pg_tblspc")) != NULL)
        {
-               if (strcmp(db_de->d_name, ".") == 0 ||
-                       strcmp(db_de->d_name, "..") == 0)
+               if (strcmp(spc_de->d_name, ".") == 0 ||
+                       strcmp(spc_de->d_name, "..") == 0)
                        continue;
 
-               snprintf(temp_path, sizeof(temp_path), "base/%s/%s",
-                                db_de->d_name, PG_TEMP_FILES_DIR);
+               snprintf(temp_path, sizeof(temp_path), "pg_tblspc/%s/%s",
+                                spc_de->d_name, PG_TEMP_FILES_DIR);
                RemovePgTempFilesInDir(temp_path);
        }
 
-       FreeDir(db_dir);
+       FreeDir(spc_dir);
 
        /*
         * In EXEC_BACKEND case there is a pgsql_tmp directory at the top level of
index 1fbeb1cc33316ad4f125b17b9640a8a8f1499952..e09072f8190ab4e6ea56e890ca8a5bd0ff10890e 100644 (file)
@@ -103,6 +103,7 @@ extern bool Log_disconnections;
 extern int     CommitDelay;
 extern int     CommitSiblings;
 extern char *default_tablespace;
+extern char *temp_tablespaces;
 extern bool fullPageWrites;
 
 #ifdef TRACE_SORT
@@ -1967,6 +1968,16 @@ static struct config_string ConfigureNamesString[] =
                "", assign_default_tablespace, NULL
        },
 
+       {
+               {"temp_tablespaces", PGC_USERSET, CLIENT_CONN_STATEMENT,
+                       gettext_noop("Sets the tablespace(s) to use for temporary tables and sort files."),
+                       NULL,
+                       GUC_LIST_INPUT | GUC_LIST_QUOTE
+               },
+               &temp_tablespaces,
+               "", assign_temp_tablespaces, NULL
+       },
+
        {
                {"default_transaction_isolation", PGC_USERSET, CLIENT_CONN_STATEMENT,
                        gettext_noop("Sets the transaction isolation level of each new transaction."),
index e7d8e41b0e1432f8254a17c47674318bf5c649d0..c56f2fd7092799c1c5cbe35a24be05b489a5f62d 100644 (file)
 #search_path = '"$user",public'                # schema names
 #default_tablespace = ''               # a tablespace name, '' uses
                                        # the default
+#temp_tablespaces = ''                 # a list of tablespace names,
+                                       # '' uses only default tablespace
 #check_function_bodies = on
 #default_transaction_isolation = 'read committed'
 #default_transaction_read_only = off
index d6a4b555bbd8336df074012ae6dd3235f4be3263..ff8221a310cd2afbae3dfbf5d9d7954d6693a0a0 100644 (file)
@@ -77,6 +77,7 @@
 
 #include "postgres.h"
 
+#include "commands/tablespace.h"
 #include "storage/buffile.h"
 #include "utils/logtape.h"
 
@@ -528,7 +529,7 @@ LogicalTapeSetCreate(int ntapes)
        Assert(ntapes > 0);
        lts = (LogicalTapeSet *) palloc(sizeof(LogicalTapeSet) +
                                                                        (ntapes - 1) *sizeof(LogicalTape));
-       lts->pfile = BufFileCreateTemp(false);
+       lts->pfile = BufFileCreateTemp(false, GetTempTablespace());
        lts->nFileBlocks = 0L;
        lts->forgetFreeSpace = false;
        lts->blocksSorted = true;       /* a zero-length array is sorted ... */
index 1cc30f4c8c1d53f3d4c36553d5ae40068f424349..0ea2ba7e2111284bf155e72534265b8b239b8370 100644 (file)
@@ -46,6 +46,7 @@
 #include "postgres.h"
 
 #include "access/heapam.h"
+#include "commands/tablespace.h"
 #include "executor/executor.h"
 #include "storage/buffile.h"
 #include "utils/memutils.h"
@@ -424,8 +425,14 @@ tuplestore_puttuple_common(Tuplestorestate *state, void *tuple)
 
                        /*
                         * Nope; time to switch to tape-based operation.
+                        *
+                        * If the temp table is slated to outlive the current transaction,
+                        * force it into my database's default tablespace, so that it will
+                        * not pose a threat to possible tablespace drop attempts.
                         */
-                       state->myfile = BufFileCreateTemp(state->interXact);
+                       state->myfile = BufFileCreateTemp(state->interXact,
+                                                                                         state->interXact ? InvalidOid :
+                                                                                         GetTempTablespace());
                        state->status = TSS_WRITEFILE;
                        dumptuples(state);
                        break;
index c5373d86eb590845cd74c61c70d787f2d452a280..1f0245448e94a7337618246e924204d036fe9240 100644 (file)
@@ -40,7 +40,8 @@ extern void AlterTableSpaceOwner(const char *name, Oid newOwnerId);
 
 extern void TablespaceCreateDbspace(Oid spcNode, Oid dbNode, bool isRedo);
 
-extern Oid     GetDefaultTablespace(void);
+extern Oid     GetDefaultTablespace(bool forTemp);
+extern Oid     GetTempTablespace(void);
 
 extern Oid     get_tablespace_oid(const char *tablespacename);
 extern char *get_tablespace_name(Oid spc_oid);
index 96ff10c94f80689aef2fbaf1531a5763fbec3241..01dd05d7b547f57d0b9b6ace2d52c733cc417322 100644 (file)
@@ -102,6 +102,8 @@ typedef struct HashJoinTableData
        BufFile   **innerBatchFile; /* buffered virtual temp file per batch */
        BufFile   **outerBatchFile; /* buffered virtual temp file per batch */
 
+       Oid                     hashTblSpc;             /* tablespace to put temp files in */
+
        /*
         * Info about the datatype-specific hash functions for the datatypes being
         * hashed. These are arrays of the same length as the number of hash join
index 309566cdfbad35da0ef3779c124ae27292cadac9..f457825b00d36fd2572263fb1f38131fe737cfa4 100644 (file)
@@ -23,7 +23,8 @@ extern TupleTableSlot *ExecHashJoin(HashJoinState *node);
 extern void ExecEndHashJoin(HashJoinState *node);
 extern void ExecReScanHashJoin(HashJoinState *node, ExprContext *exprCtxt);
 
-extern void ExecHashJoinSaveTuple(MinimalTuple tuple, uint32 hashvalue,
-                                         BufFile **fileptr);
+extern void ExecHashJoinSaveTuple(HashJoinTable hashtable,
+                                                                 MinimalTuple tuple, uint32 hashvalue,
+                                                                 BufFile **fileptr);
 
 #endif   /* NODEHASHJOIN_H */
index fd4c782368791abeb2e40b306346b79b77f87098..76ba602e9f5dcf8fdf0c1bcc0a527dd3a1caabf5 100644 (file)
@@ -34,7 +34,7 @@ typedef struct BufFile BufFile;
  * prototypes for functions in buffile.c
  */
 
-extern BufFile *BufFileCreateTemp(bool interXact);
+extern BufFile *BufFileCreateTemp(bool interXact, Oid tblspcOid);
 extern void BufFileClose(BufFile *file);
 extern size_t BufFileRead(BufFile *file, void *ptr, size_t size);
 extern size_t BufFileWrite(BufFile *file, void *ptr, size_t size);
index f2ff0bc6785368f260af127dcd0a559679d71174..6a8847db173028485bdff29a09f0f08f48fda1b0 100644 (file)
@@ -59,9 +59,8 @@ extern int    max_files_per_process;
  */
 
 /* Operations on virtual Files --- equivalent to Unix kernel file ops */
-extern File FileNameOpenFile(FileName fileName, int fileFlags, int fileMode);
 extern File PathNameOpenFile(FileName fileName, int fileFlags, int fileMode);
-extern File OpenTemporaryFile(bool interXact);
+extern File OpenTemporaryFile(bool interXact, Oid tblspcOid);
 extern void FileClose(File file);
 extern void FileUnlink(File file);
 extern int     FileRead(File file, char *buffer, int amount);
index 15c4e6b4844d2fa553a6be1578ecdb62e49a6935..7751ce9c430ddffdffcb396941efe0584dd93ae3 100644 (file)
@@ -225,6 +225,8 @@ extern void read_nondefault_variables(void);
 /* in commands/tablespace.c */
 extern const char *assign_default_tablespace(const char *newval,
                                                  bool doit, GucSource source);
+extern const char *assign_temp_tablespaces(const char *newval,
+                                                 bool doit, GucSource source);
 
 /* in utils/adt/regexp.c */
 extern const char *assign_regex_flavor(const char *value,