Content-Type: text/plain;
	charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3774.500.171.1.1\))
Subject: Re: open items
From: Daniel Gustafsson <daniel@yesql.se>
In-Reply-To: <CAAKRu_bfeF0mdiCe0VJ38mjNMpH=ZEp4yLG1yzUGp=3VWaQsOg@mail.gmail.com>
Date: Fri, 10 May 2024 15:28:03 +0200
Cc: Robert Haas <robertmhaas@gmail.com>,
 PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <FB16787F-BD81-4208-AE0F-B802895E5904@yesql.se>
References: <CA+TgmoaZqRyVLNutg3Q0n=2p+SqoWopYcMv6D=bTiyq2-fSG0Q@mail.gmail.com>
 <CAAKRu_bfeF0mdiCe0VJ38mjNMpH=ZEp4yLG1yzUGp=3VWaQsOg@mail.gmail.com>
To: Melanie Plageman <melanieplageman@gmail.com>
Archived-At: <https://www.postgresql.org/message-id/FB16787F-BD81-4208-AE0F-B802895E5904%40yesql.se>
Precedence: bulk

> On 10 May 2024, at 14:48, Melanie Plageman <melanieplageman@gmail.com> =
wrote:
>=20
> On Thu, May 9, 2024 at 3:28=E2=80=AFPM Robert Haas =
<robertmhaas@gmail.com> wrote:
>>=20
>> Just a few reminders about the open items list at
>> https://wiki.postgresql.org/wiki/PostgreSQL_17_Open_Items --
>>=20
>> * Incorrect Assert in heap_end/rescan for BHS. Either the description
>> of this item is inaccurate, or we've been unable to fix an incorrect
>> assert after more than a month. I interpret
>> =
https://www.postgresql.org/message-id/54858BA1-084E-4F7D-B2D1-D15505E512FF=
%40yesql.se
>> as a vote in favor of committing some patch by Melanie to fix this.

It's indeed a vote for that.

>> Either Tomas should commit that patch, or Melanie should commit that
>> patch, or somebody should say why that patch shouldn't be committed,
>> or someone should request more help determining whether that patch is
>> indeed the correct fix, or something. But let's not just sit on this.
>=20
> Sorry, yes, the trivial fix has been done for a while. There is one
> outstanding feedback on the patch: an update to one of the comments
> suggested by Tomas. I got distracted by trying to repro and fix a bug
> from the section "live issues affecting stable branches". I will
> update this BHS patch by tonight and commit it once Tomas has a chance
> to +1.

+1

--
Daniel Gustafsson