Re: Errors creating partitioned tables from existing using (LIKE <table>) after renaming table constraints

From: Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Stuart <sfbarbee(at)gmail(dot)com>, pgsql-bugs(at)postgresql(dot)org
Subject: Re: Errors creating partitioned tables from existing using (LIKE <table>) after renaming table constraints
Date: 2018-12-14 02:19:16
Message-ID: d6975ae7-e82a-75d7-fe35-91145cb6265b@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On 2018/12/14 9:20, Michael Paquier wrote:
> On Thu, Dec 13, 2018 at 06:03:35PM +0900, Amit Langote wrote:
>> What's happening here is that when the ALTER TABLE RENAME CONSTRAINT is
>> followed by CREATE TABLE (LIKE .. INCLUDING ALL) in the same session, the
>> latter is referring to *stale* information about constraints of the source
>> table. You said it works correctly after you drop and re-create the
>> constraint, but that's only because ALTER TABLE DROP/ADD CONSTRAINT will
>> correctly invalidate the cached information, so that subsequent CREATE
>> TABLE sees the correct information from the updated cache. The way to fix
>> it is to teach ALTER TABLE RENAME CONSTRAINT to reset the cached
>> information.
>
> This analysis looks right to me, and that's indeed a bug. And as far as
> I can see this is reproducible down to 9.4. I cannot check your patch
> in details today unfortunately, but I'll try to look at that in the next
> couple of days and see if there are any surrounding issues.

Thank you for looking. I noticed that the previously posted patch doesn't
apply as is to branches before 11, so here are the patches that apply to
9.4 to 10 branches.

Regards,
Amit

Attachment Content-Type Size
pg10-0001-Make-rename_constraint_internal-invalidate-relcache.patch text/plain 3.2 KB
pg94-0001-Make-rename_constraint_internal-invalidate-relcache.patch text/plain 3.0 KB
pg95-0001-Make-rename_constraint_internal-invalidate-relcache.patch text/plain 3.1 KB
pg96-0001-Make-rename_constraint_internal-invalidate-relcache.patch text/plain 3.1 KB

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Amit Langote 2018-12-14 04:22:26 Re: BUG #15212: Default values in partition tables don't work as expected and allow NOT NULL violation
Previous Message Amit Langote 2018-12-14 00:42:09 Re: create partitioned table with (like table INCLUDING ALL ) fails with "insufficient columns in UNIQUE constraint definition"