Re: pgsql: Allow multiple xacts during table sync in logical replication. - Mailing list pgsql-committers

From Amit Kapila
Subject Re: pgsql: Allow multiple xacts during table sync in logical replication.
Date
Msg-id CAA4eK1LkG8fQ+Bd8DbGLjNOtX_mteTGqPF2vo-YRS4KVZsnh0w@mail.gmail.com
Whole thread Raw
In response to Re: pgsql: Allow multiple xacts during table sync in logical replication.  (Amit Kapila <amit.kapila16@gmail.com>)
List pgsql-committers
On Sat, Feb 13, 2021 at 10:56 AM Amit Kapila <amit.kapila16@gmail.com> wrote:
>
> On Sat, Feb 13, 2021 at 9:49 AM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> >
> > Amit Kapila <amit.kapila16@gmail.com> writes:
> > > Attached should fix the reported warnings but as I am not getting
> > > those warnings on my system so can't confirm but otherwise, the patch
> > > works as expected. I am planning to push this unless you or Peter have
> > > any comments.
> >
> > Personally I'd get rid of the option for ReplicationSlotNameForTablesync
> > to allocate the result buffer.
> >
>
> Sounds reasonable and the updated patch attached.
>

Pushed and I don't see those warnings on caiman.

-- 
With Regards,
Amit Kapila.



pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Make ExecGetInsertedCols() and friends more robust and improve c
Next
From: Tom Lane
Date:
Subject: pgsql: Simplify loop logic in nodeIncrementalSort.c.