Re: Batch TIDs lookup in ambulkdelete

From: Matheus Alcantara <matheusssilv97(at)gmail(dot)com>
To: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Batch TIDs lookup in ambulkdelete
Date: 2025-06-09 22:24:23
Message-ID: CAFY6G8dT83G5yvEJW=HrAZPaNQK5o=nJ-9neQFtrbsmSa5+Qcw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 06/06/25 18:34, Masahiko Sawada wrote:
>
> Thank you for the report. With the 0001 and 0002 patches, I got a
> SEGV. I've fixed this issue in the attached updated version patches.
> I've confirmed that the patches pass CI tests but I'm not sure it
> fixes the shared memory segment leak problem you reported. The
> attached patches incorporated the comments[1] from John as well.
>

Thanks for the new version. I've tested the v3 version attached on [1]
and I can confirm that the tests are now passing and I can't see the
shared memory issues that I've mentioned on v1.

Thanks!

[1] https://www.postgresql.org/message-id/CAD21AoAcfp5kdcsT5727Vw4JF-Rw7b73zXh_GwXqNHg3P7-UoA%40mail.gmail.com

--
Matheus Alcantara

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Dmitry Koval 2025-06-09 22:48:33 Re: Add SPLIT PARTITION/MERGE PARTITIONS commands
Previous Message Melanie Plageman 2025-06-09 22:22:00 Re: Periodic FSM vacuum doesn't happen in one-pass strategy vacuum.