RE: Conflict detection for update_deleted in logical replication

From: "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com>
To: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: shveta malik <shveta(dot)malik(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Nisha Moond <nisha(dot)moond412(at)gmail(dot)com>, "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: RE: Conflict detection for update_deleted in logical replication
Date: 2025-05-26 07:16:29
Message-ID: OS0PR01MB57161C2C4345AAFB3EA2E5309465A@OS0PR01MB5716.jpnprd01.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, May 23, 2025 at 7:08 PM Amit Kapila wrote:

>
> On Thu, May 22, 2025 at 8:28 AM Zhijie Hou (Fujitsu)
> <houzj(dot)fnst(at)fujitsu(dot)com> wrote:
> >
> > Attaching the V31 patch set which addressed comments in [1]~[8].
> >
>
> Few comments:
>
> Apart from the above comments, I made a few other cosmetic changes in
> the attached.

Thanks for the comments and diff, I have merged them into the latest patch set.

Attaching the V32 patch set which addressed comments in [1]~[5].

[1] https://www.postgresql.org/message-id/CAFiTN-vd%3DrF0CK1sw-vdOAYSRaV_GMrO3VR0yTv%3Df2GKyOw%3Dbg%40mail.gmail.com
[2] https://www.postgresql.org/message-id/CAA4eK1Kd05tu9n%2Ba4LAmK-UWxTfcaanuLYD2e1Lt-2kiKzvwTw%40mail.gmail.com
[3] https://www.postgresql.org/message-id/CABPTF7U8SKpw%2BHepd4MJ%2BoPBAWdFmYukyk92TKLkZGHhQD4KaQ%40mail.gmail.com
[4] https://www.postgresql.org/message-id/CAA4eK1J%2BdmsB%3Dod4pkGBjZkeurpWGuRXOeXBbNRbh3hVUr7DxQ%40mail.gmail.com
[5] https://www.postgresql.org/message-id/CAJpy0uDEKO%2BqX1Y7WQar_o6nCPBQtenjG%3D8fE2NvKcwtFRs6hw%40mail.gmail.com

Best Regards,
Hou zj

Attachment Content-Type Size
v32-0006-Add-a-tap-test-to-verify-the-management-of-the-n.patch application/octet-stream 6.7 KB
v32-0002-Maintain-the-replication-slot-in-logical-launche.patch application/octet-stream 19.9 KB
v32-0003-Add-a-retain_conflict_info-option-to-subscriptio.patch application/octet-stream 87.7 KB
v32-0004-Introduce-a-new-GUC-max_conflict_retention_durat.patch application/octet-stream 29.5 KB
v32-0005-Re-create-the-replication-slot-if-the-conflict-r.patch application/octet-stream 10.8 KB
v32-0007-Support-the-conflict-detection-for-update_delete.patch application/octet-stream 23.7 KB
v32-0001-Maintain-the-oldest-non-removable-transaction-ID.patch application/octet-stream 44.8 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Zhijie Hou (Fujitsu) 2025-05-26 07:16:39 RE: Conflict detection for update_deleted in logical replication
Previous Message Amit Kapila 2025-05-26 06:49:00 Re: Slot's restart_lsn may point to removed WAL segment after hard restart unexpectedly