RE: Conflict detection for update_deleted in logical replication

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

On Wednesday, August 27, 2025 11:30 AM Zhijie Hou (Fujitsu) <houzj(dot)fnst(at)fujitsu(dot)com> wrote:
>
> Here is V66 patch set which includes the following changes:
>
> [0001]
> * Enhanced documentation according to Dilip's comments [1].
> * Simplified logic by directly using the leader's oldest_nonremovable_xid to
> locate deleted tuples, according to Amit's comments [2].
> * Merged Amit's diff [2].
> * Integrated the new NOTICE into the existing CheckSubDeadTupleRetention
> function according to Amit's feedback [3].
> * Some other adjustments according to feedback from [2] [3].
>
> [0002]
> * Refactored code according to Shveta's comments [4].
> * Removed an unnecessary variable according to Shveta's suggestions [5].
> * Some other adjustments according to feedback from [4][5][6].

I noticed that Cfbot failed to compile the document due to a typo after renaming
the subscription option. Here are the updated V67 patches to fix that, only the doc
in 0001 is modified.

Best Regards,
Hou zj

Attachment Content-Type Size
v67-0003-Add-a-dead_tuple_retention_active-column-in-pg_s.patch application/octet-stream 8.1 KB
v67-0001-Introduce-a-max_retention_duration-option-to-sub.patch application/octet-stream 101.1 KB
v67-0002-Resume-retaining-the-information-for-conflict-de.patch application/octet-stream 19.2 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Hayato Kuroda (Fujitsu) 2025-08-28 02:45:03 RE: POC: enable logical decoding when wal_level = 'replica' without a server restart
Previous Message Chao Li 2025-08-28 02:30:13 Re: Mark ItemPointer parameters as const in tuple/table lock functions