Re: Avoiding another needless ERROR during nbtree page deletion

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Avoiding another needless ERROR during nbtree page deletion
Date: 2023-05-25 22:40:16
Message-ID: CAH2-Wz=fQKiFAPryM6VsL2VpOh=zB_AMEY7GLGPdNk3gbyQD7A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, May 22, 2023 at 10:59 AM Peter Geoghegan <pg(at)bowt(dot)ie> wrote:
> Attached is v2, which does it that way. It also adjusts the approach
> taken to release locks and pins when the left sibling validation check
> fails.

I pushed this just now, backpatching all the way.

> Not including a revised amcheck patch here, since I'm not exactly sure
> what to do with your feedback on that one just yet.

I'd like to go with a minimal approach in my patch to address the
remaining issue in amcheck. Something similar to the patch that was
posted as part of v1. While it seems important to address the issue,
making sure that we have coverage of the leftmost page really being
half-dead (as opposed to something that would constitute corruption)
seems much less important. Ideally we'd have exhaustive coverage, but
it's not a priority for me right now.

--
Peter Geoghegan

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2023-05-25 23:47:16 Re: Cutting support for OpenSSL 1.0.1 and 1.0.2 in 17~?
Previous Message Michael Paquier 2023-05-25 22:39:17 Re: Implement generalized sub routine find_in_log for tap test