pgsql: pg_upgrade: check for inconsistencies in not-null constraints w/ - Mailing list pgsql-committers

From Álvaro Herrera
Subject pgsql: pg_upgrade: check for inconsistencies in not-null constraints w/
Date
Msg-id E1uXiwH-004h95-0S@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
pg_upgrade: check for inconsistencies in not-null constraints w/inheritance

With tables defined like this,
  CREATE TABLE ip (id int PRIMARY KEY);
  CREATE TABLE ic (id int) INHERITS (ip);
  ALTER TABLE ic ALTER id DROP NOT NULL;

pg_upgrade fails during the schema restore phase due to this error:
  ERROR: column "id" in child table must be marked NOT NULL

This can only be fixed by marking the child column as NOT NULL before
the upgrade, which could take an arbitrary amount of time (because ic's
data must be scanned).  Have pg_upgrade's check mode warn if that
condition is found, so that users know what to adjust before running the
upgrade for real.

Author: Ali Akbar <the.apaan@gmail.com>
Reviewed-by: Justin Pryzby <pryzby@telsasoft.com>
Backpatch-through: 13
Discussion: https://postgr.es/m/CACQjQLoMsE+1pyLe98pi0KvPG2jQQ94LWJ+PTiLgVRK4B=i_jg@mail.gmail.com

Branch
------
REL_17_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/b8b2e60527c67599bb74895fc99d7feb7f97d706

Modified Files
--------------
src/bin/pg_upgrade/check.c | 86 ++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 86 insertions(+)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: Disable commit timestamps during bootstrap
Next
From: Álvaro Herrera
Date:
Subject: pgsql: Fix new pg_upgrade query not to rely on regnamespace