Re: Should new partitions inherit their tablespace from their parent?

From: David Rowley <david(dot)rowley(at)2ndquadrant(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Should new partitions inherit their tablespace from their parent?
Date: 2018-12-17 01:40:53
Message-ID: CAKJS1f_gz85CgNbcych11oQGGi-=dxkapL5cb04pANpePr-zkA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, 17 Dec 2018 at 12:59, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> Okay, I think that you should add an assertion on
> CheckRelationLockedByMe() as MergeAttributes()'s only caller is
> DefineRelation(). Better safe than sorry later.

Would that not just double up the work that's already done in relation_open()?

--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2018-12-17 02:01:59 Re: Catalog views failed to show partitioned table information.
Previous Message Amit Langote 2018-12-17 01:22:28 Re: Catalog views failed to show partitioned table information.