Re: PATCH: default_index_tablespace

From: Greg Stark <stark(at)mit(dot)edu>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: jltallon(at)adv-solutions(dot)net, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: PATCH: default_index_tablespace
Date: 2015-04-16 09:17:57
Message-ID: CAM-w4HPOASwsQMdGZqjyFHNubbUnWrUAo8ibci-97UKU=poDbg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 15 Apr 2015 19:12, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> I'm afraid this idea is a nonstarter, because it will break existing
> applications, and in particular existing pg_dump output files, which
> expect to be able to determine an index's tablespace by setting
> "default_tablespace". (It is *not* adequate that the code falls back
> to "default_tablespace" if the new GUC is unset; if it is set, you've
> still broken pg_dump.) The incremental value, if indeed there is any,
> of being able to control index positioning this way seems unlikely to
> justify a backwards-compatibility break of such magnitude.

Just brainstorming here but that just means "default_tablespace" needs to
take precedence. We could have a default_table_tablespace and
default_index_tablespace which default_tablespace overrides. Or we could
allow a mini config language in default_tablespace like
"table=space1,index=space2".

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2015-04-16 09:18:37 Re: INSERT ... ON CONFLICT IGNORE (and UPDATE) 3.0
Previous Message Asif Naeem 2015-04-16 08:40:09 Re: Fix broken Install.bat when target directory contains a space