Re: [Patch] add new parameter to pg_replication_origin_session_setup - Mailing list pgsql-hackers

From shveta malik
Subject Re: [Patch] add new parameter to pg_replication_origin_session_setup
Date
Msg-id CAJpy0uB-orRxTL-8MJ+i9riBZkvVdP_T+p71GyHnJRYw-MrXVg@mail.gmail.com
Whole thread
In response to Re: [Patch] add new parameter to pg_replication_origin_session_setup  (Amit Kapila <amit.kapila16@gmail.com>)
Responses Re: [Patch] add new parameter to pg_replication_origin_session_setup
List pgsql-hackers
On Wed, Mar 25, 2026 at 1:51 PM Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> On Wed, Feb 11, 2026 at 5:09 PM shveta malik <shveta.malik@gmail.com> wrote:
> >
> > On Wed, Feb 11, 2026 at 3:41 PM Amit Kapila <amit.kapila16@gmail.com> wrote:
> > >
> > >
> > > I see your point but one advantage with the proposed code change is
> > > that it started to appear that we can extend this part of code easily
> > > in the future as it separates most of the handling related to when a
> > > user has given acquired_by parameter's value as zero and non-zero.
> >
> > Okay, yes. So I am okay with it. The slight change I suggested (if to
> > else-if) and a comment will make it more clean.
> >
>
> I have tried to address both your suggestions in the attached. See, if
> this looks okay to you now?
>

LGTM now, thanks!

thanks
Shveta



pgsql-hackers by date:

Previous
From: Jim Jones
Date:
Subject: Re: COMMENTS are not being copied in CREATE TABLE LIKE
Next
From: Peter Eisentraut
Date:
Subject: Re: SQL Property Graph Queries (SQL/PGQ)