Re: BUG #18550: Cross-partition update of a former inheritance parent leads to an assertion failure

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: exclusion(at)gmail(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #18550: Cross-partition update of a former inheritance parent leads to an assertion failure
Date: 2024-07-24 10:44:50
Message-ID: 202407241044.4auhbvwjgta2@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On 2024-Jul-23, Tom Lane wrote:

> Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:

> > Yeah, that seems to work. We can do that essentially for free in
> > StorePartitionBound(), like in the attached (which was taken in 14).
>
> Looks sane.

Thanks for looking.

> > I'd add Alexander's script as a test too, somewhere near the bottom of
> > create_table.sql.
>
> Personally I'd include this in the existing alter_table.sql test case,
> to save a few steps:

Nice! Done that way.

I didn't see the connection between the assertion failure and
8f4a6b9e4f5b, but I didn't spend much time on it. But I didn't see any
reason not to backpatch this all the way down, either, so I did that.

Thanks, Alexander, for reporting this.

--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
<Schwern> It does it in a really, really complicated way
<crab> why does it need to be complicated?
<Schwern> Because it's MakeMaker.

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Masahiko Sawada 2024-07-24 21:51:20 Re: "unexpected duplicate for tablespace" problem in logical replication
Previous Message Johannes Ernst 2024-07-23 22:18:15 Re: BUG #18549: Support ipv6 link-local addresses in the inet type