From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Jehan-Guillaume de Rorthais <jgdr(at)dalibo(dot)com> |
Cc: | pgsql-bugs(at)lists(dot)postgresql(dot)org |
Subject: | Re: FK violation in partitioned table after truncating a referenced partition |
Date: | 2020-02-07 17:27:51 |
Message-ID: | 20200207172751.GA26691@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
On 2020-Feb-07, Jehan-Guillaume de Rorthais wrote:
> Maybe I would just add:
>
> /*
> * If this constraint has a parent constraint which we have not seen
> * yet, keep track of it for the second loop, below.
> + * Tracking parent constraint allows to climb up to the top-level
> + * level constraint and look for all possible relation referencing
> + * the partioned table.
> */
LGTM.
BTW I was thinking that perhaps it would make sense to go up all levels
at once when we see a "parented" constraint; this would avoid having to
restart several times when there's N-levels partitioning. It might be
an issue if pg_constraint is large, because, you see, there's a seqscan
there! (Maybe now's the time to add an index to confrelid, but of
course only in master). This probably doesn't matter much normally
because nobody uses that many partition levels ...
> This is out of the scope of this bug fix in my humble opinion. This would be a
> whole new feature, even if it could be done without a new syntax.
Sure.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Adam Middleton | 2020-02-07 17:37:52 | Southern California 2020 Linux Expo Emails |
Previous Message | Jehan-Guillaume de Rorthais | 2020-02-07 16:19:33 | Re: FK violation in partitioned table after truncating a referenced partition |