agora inbox for pgsql-bugs@postgresql.org  
help / color / mirror / Atom feed
From: Álvaro Herrera <alvherre@kurilemu.de>
To: Japin Li <japinli@hotmail.com>
Cc: Keith Paskett <keith.paskett@logansw.com>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: pg_restore error with partitioned table having exclude constraint
Date: Mon, 28 Apr 2025 10:42:14 +0200
Message-ID: <202504280842.2jaxgkldoap2@alvherre.pgsql> (raw)
In-Reply-To: <ME0P300MB04459912DFAA3BEC7215C0BEB6852@ME0P300MB0445.AUSP300.PROD.OUTLOOK.COM>

On 2025-Apr-24, Japin Li wrote:

> On Thu, 17 Apr 2025 at 17:18, Álvaro Herrera <alvherre@kurilemu.de> wrote:
> > On 2025-Apr-17, Japin Li wrote:
> >
> >> It seems PG 16 does not support exclusion constraints on
> >> partitioned tables.
> >
> > Yeah, my recollection is that they were purposefully disallowed
> > (mainly because I didn't want to research how to fully make them
> > work when adding local partitioned indexes), and that we needed to
> > do more work if we wanted to let them through.  I suspect commit
> > 8c852ba9a4 was mistaken to allow that case without looking for
> > further implications.
> 
> Sorry, I’m unclear on “more work.” Can you explain further?

Well, there are no tests in the patch.  8c852ba9a434 added some, but
it's now clear that something was overlooked.  I think this patch should
make more of an effort to cover all interesting cases in regression
tests if there are holes in coverage; and also add something to verify
that pg_dump and pg_upgrade work correctly for these constraints.

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/
"Si quieres ser creativo, aprende el arte de perder el tiempo"





view thread (15+ messages)  latest in thread

Message-ID: <202504280842.2jaxgkldoap2@alvherre.pgsql>
Permalink:  ../202504280842.2jaxgkldoap2@alvherre.pgsql/
Also on:    postgresql.org/message-id/202504280842.2jaxgkldoap2@alvherre.pgsql

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-bugs@postgresql.org
  Cc: alvherre@kurilemu.de, japinli@hotmail.com, keith.paskett@logansw.com, pgsql-bugs@lists.postgresql.org
  Subject: Re: pg_restore error with partitioned table having exclude constraint
  In-Reply-To: <202504280842.2jaxgkldoap2@alvherre.pgsql>

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

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