Re: behaviour change - default_tablesapce + partition table - Mailing list pgsql-hackers

From Amit Langote
Subject Re: behaviour change - default_tablesapce + partition table
Date
Msg-id 170fcf53-d79b-c3a2-3d9a-46e3dfc19af2@lab.ntt.co.jp
Whole thread Raw
In response to behaviour change - default_tablesapce + partition table  (Rushabh Lathia <rushabh.lathia@gmail.com>)
Responses Re: behaviour change - default_tablesapce + partition table  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-hackers
On 2019/05/17 12:40, Rushabh Lathia wrote:
> Hi,
> 
> Consider the below test:
> 
> create tablespace mytbs location '/home/rushabh/mywork/workspace/pg/';
> create table test ( a int , b int ) partition by list (a);
> 
> set default_tablespace to mytbs;
> create table test_p1 partition of test for values in (1);
> 
> In the above test, after the setting the default_tablespace I am creating
> a partition table and expecting that to get created under "mytbs"
> tablespace.
> 
> But that is not the case:
> 
> postgres@66247=#select relname, reltablespace from pg_class where relname =
> 'test_p1';
>  relname | reltablespace
> ---------+---------------
>  test_p1 |             0
> (1 row)
> 
> I noticed the behaviour change for default_tablespace with partition table
> with below commit.
> 
> commit 87259588d0ab0b8e742e30596afa7ae25caadb18
> Author: Alvaro Herrera <alvherre@alvh.no-ip.org>
> Date:   Thu Apr 25 10:20:23 2019 -0400
> 
>     Fix tablespace inheritance for partitioned rels
> 
>     Commit ca4103025dfe left a few loose ends.  The most important one
>     (broken pg_dump output) is already fixed by virtue of commit
>     3b23552ad8bb, but some things remained:
> 
> I don't think that new behaviour is intended

Should we add this to open items?

Thanks,
Amit




pgsql-hackers by date:

Previous
From: Thomas Munro
Date:
Subject: Parallel Append subplan order instability on aye-aye
Next
From: Thomas Munro
Date:
Subject: Re: Do we expect tests to work with default_transaction_isolation=serializable