Re: Conflict Detection and Resolution

From: Ajin Cherian <itsajin(at)gmail(dot)com>
To: Peter Smith <smithpb2250(at)gmail(dot)com>
Cc: Nisha Moond <nisha(dot)moond412(at)gmail(dot)com>, shveta malik <shveta(dot)malik(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, Jan Wieck <jan(at)wi3ck(dot)info>, Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com>
Subject: Re: Conflict Detection and Resolution
Date: 2024-10-21 00:45:20
Message-ID: CAFPTHDbr-MKkBfzecTbB1hD=7NB9c1-5716UNXvzKNp0K0i3Qg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Oct 15, 2024 at 6:00 PM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> Hi Ajin/Nisha -- Here are my review comments for patch v15-0001 (code).
>
> (AFAIK v16-0001 is the same as v15-0001, so this review is up to date)
>
> Please also see the "nits" attachment to this post, which has many
> more review comments of a more cosmetic nature.
>

Here's patch v17 addressing all review comments from Peter and Shveta:

Thanks Nisha for working on patches 0003 and 0004.

Patches:
- https://www.postgresql.org/message-id/CAJpy0uAEfA9Bhi%3D8U6dP--Jovj4ZZK88x2upD06xV1eMEAGhxw%40mail.gmail.com
- https://www.postgresql.org/message-id/CAJpy0uAf9DugEeat0SASnch7LHyeJxsvoMrvoH08uBbOcKFSqQ%40mail.gmail.com
- https://www.postgresql.org/message-id/CAHut%2BPsDH-%3DRMsemWau%2B_DL1sLrMc5DjQpLTgek22UW%3D1jPayg%40mail.gmail.com

Attachment Content-Type Size
v17-0004-Implements-last_update_wins-conflict-resolver.patch application/octet-stream 28.0 KB
v17-0003-Conflict-resolution-for-update_exists-conflict-t.patch application/octet-stream 21.2 KB
v17-0001-Add-CONFLICT-RESOLVERS-into-the-syntax-for-CREAT.patch application/octet-stream 108.3 KB
v17-0002-Conflict-resolvers-for-insert-update-and-delete.patch application/octet-stream 72.9 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michail Nikolaev 2024-10-21 01:14:00 Re: Conflict Detection and Resolution
Previous Message Fujii Masao 2024-10-21 00:19:49 Refactor GetLockStatusData() by skipping unused backends and groups