pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Álvaro Herrera <alvherre@kurilemu.de>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Fix determination of not-null constraint "locality" for inherite
Date: Sat, 18 Oct 2025 16:19:22 +0000
Message-ID: <E1vA9eM-002KBg-0Q@gemulon.postgresql.org> (raw)

Fix determination of not-null constraint "locality" for inherited columns

It is possible to have a non-inherited not-null constraint on an
inherited column, but we were failing to preserve such constraints
during pg_upgrade where the source is 17 or older, because of a bug in
the pg_dump query for it.  Oversight in commit 14e87ffa5c54.  Fix that
query.  In passing, touch-up a bogus nearby comment introduced by the
same commit.

In version 17, make the regression tests leave a table in this
situation, so that this scenario is tested in the cross-version upgrade
tests of 18 and up.

Author: Dilip Kumar <dilipbalaut@gmail.com>
Reported-by: Andrew Bille <andrewbille@gmail.com>
Bug: #19074
Backpatch-through: 18
Discussion: https://postgr.es/m/19074-ae2548458cf0195c@postgresql.org

Branch
------
REL_17_STABLE

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

Modified Files
--------------
src/test/regress/expected/constraints.out | 4 ++++
src/test/regress/sql/constraints.sql      | 5 +++++
2 files changed, 9 insertions(+)



view thread (3+ messages)  latest in thread

Message-ID: <E1vA9eM-002KBg-0Q@gemulon.postgresql.org>
Permalink:  ../E1vA9eM-002KBg-0Q@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1vA9eM-002KBg-0Q@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: alvherre@kurilemu.de, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Fix determination of not-null constraint "locality" for inherite
  In-Reply-To: <E1vA9eM-002KBg-0Q@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox