On Wed, Jun 30, 2021 at 9:29 AM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Tue, Jun 29, 2021 at 12:57 PM Dilip Kumar <dilipbal...@gmail.com> wrote: > > > > On Fri, Jun 25, 2021 at 12:24 PM Amit Kapila <amit.kapil...@gmail.com> > > wrote: > > > > > > Till now, we didn't allow to stream the changes in logical replication > > > till we receive speculative confirm or the next DML change record > > > after speculative inserts. The reason was that we never use to process > > > speculative aborts but after commit 4daa140a2f it is possible to > > > process them so we can allow streaming once we receive speculative > > > abort after speculative insertion. See attached. > > > > > > I think this is a minor improvement in the logical replication of > > > in-progress transactions. I have verified this for speculative aborts > > > and it allows streaming once we receive the spec_abort change record. > > > > Yeah, this improvement makes sense. And the patch looks fine to me. > > > > Thanks. Now, that the PG-15 branch is created, I think we should > commit this to both 15 and 14 as this is a minor change. What do you > think?
Yeah, this is a minor improvement so can be pushed to both 15 and 14. -- Regards, Dilip Kumar EnterpriseDB: http://www.enterprisedb.com
