On Tue, 20 Jan 2026 at 15:30, Andrey Borodin <[email protected]> wrote:
>
>
>
> > On 15 Jan 2026, at 22:59, Kirill Reshke <[email protected]> wrote:
> >
> > PFA v2 which leaves the test in-place.
> >
> > Also commit message improvements.
>
> Yeah, killtuples for GiST root page is broken. Your patch is fixing it.
> I don't think we should backpatch this, the bug is harmless, but for master 
> the patch LGTM.

Thank you

> It would be good to assign so->curBlkno and so->curBlkno together. But 
> gistScanPage() is the only place with access to the block number.

Sorry, didnt get this take.

> +# Test gist, but with fewer rows - that killitems used to be buggy.
>
> Probably, in this comment we can explicitly say that killitems was buggy, but 
> now is fixed.
>

Hmm, what would be a good wording here?


-- 
Best regards,
Kirill Reshke


Reply via email to