Re: ALTER TABLE validate foreign key dependency problem
David Rowley <dgrowleyml@gmail.com>
From: David Rowley <dgrowleyml@gmail.com>
To: Simon Riggs <simon@2ndquadrant.com>
Cc: PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2020-07-14T05:10:25Z
Lists: pgsql-hackers
On Mon, 13 Jul 2020 at 08:13, Simon Riggs <simon@2ndquadrant.com> wrote: > > On Sun, 12 Jul 2020 at 05:51, David Rowley <dgrowleyml@gmail.com> wrote: > >> >> > I also considered that we could just delay all foreign key validations >> > until phase 3, but I ended up just doing then only when a rewrite is >> > pending. >> >> I still wonder if it's best to delay the validation of the foreign key >> regardless of if there's a pending table rewrite, but the patch as it >> is now only delays if there's a pending rewrite. > > > Consistency seems the better choice, so I agree we should validate later in all cases. Does changing that have any other effects? Thanks for having a look here. I looked at this again and noticed it wasn't just FOREIGN KEY constraints. CHECK constraints were being validated at the wrong time too. I did end up going with unconditionally moving the validation until phase 3. I've pushed fixed back to 9.5 David
Commits
-
Fix timing issue with ALTER TABLE's validate constraint
- 1231a0b0ea66 12.4 landed
- e20003fa0f8d 9.5.23 landed
- c732df133a3d 9.6.19 landed
- 4e02f88a4eed 10.14 landed
- d2761b680a90 11.9 landed
- b827304291af 13.0 landed
- f1fcf2d3b2e0 14.0 landed