On 1/13/07, Albert Cervera Areny <albertca(at)hotpop(dot)com> wrote:
> It was already possible to set the guc on postgresql.conf when I posted the
ok... fixed... the problem was that this code only let
num_temp_tablespaces be greater than zero when we are in an
interactive command (eg. a SET command) but setting the guc from
postgresql.conf at startup time is not interactive so
num_temp_tablespaces is zero and when i try to get the first temp
tablespace to use (MyProcPid % num_temp_tablespaces) causes a floatin
exception (division by zero).
+ if (source >= PGC_S_INTERACTIVE && IsTransactionState())
+ * Verify that all the names are valid tablspace names
+ * We do not check for USAGE rights should we?
+ foreach(l, namelist)
+ char *curname = (char *) lfirst(l);
+ if (get_tablespace_oid(curname) == InvalidOid)
+ ereport((source == PGC_S_TEST) ? NOTICE : ERROR,
+ errmsg("tablespace \"%s\" does not exist", curname)));
new patch added, with that piece of code refactored to let
num_temp_tablespaces get a value greater than zero always that the guc
is setted, i also add some docs.
the patch passes all 104 regression tests and all my tests as well...
i think the patch is ready to be applied to HEAD, any committer want
to review it?
"Programming today is a race between software engineers striving to
build bigger and better idiot-proof programs and the universe trying
to produce bigger and better idiots.
So far, the universe is winning."
In response to
pgsql-patches by date
|Next:||From: ITAGAKI Takahiro||Date: 2007-01-15 08:36:09|
|Subject: O_DIRECT support for Windows|
|Previous:||From: Matthew T. O'Connor||Date: 2007-01-14 22:49:27|
|Subject: Re: Autovacuum improvements|