No worries, Jake. The steps you are proposing sound good to me. Let me know if I can be of any help.
Alberto ________________________________ From: Jacob Barrett <jabarr...@vmware.com> Sent: Wednesday, September 22, 2021 7:44 PM To: dev@geode.apache.org <dev@geode.apache.org> Subject: Re: PROPOSAL: Remove WAN TX Batching Serialization Changes > On Sep 22, 2021, at 12:31 AM, Alberto Gomez <alberto.go...@est.tech> wrote: > > Hi, > > Jake, why do you say the feature is not complete in 1.14.0? In my view, it > works as it was designed and as documented. Sorry, I was misinformed and misjudge the state. I meant no disrespect. > I do not think it makes sense to remove a feature shipped in the 1.14.0 > release that customers might already be using. I agree! > If the urgent issue to solve is the unnecessary serialization overhead when > the WAN or AEQ events are part of a transaction and the sender has not > enabled TX batching, I propose we try to fix just this in 1.14.1. Yes, let me look at an alternative approach for 1.14 that can reduce that overhead when the sender is not batching transactions. I think it makes send to scratch the original proposal. I will just write up a JIRA and PR for the protocol changes agains develop and then we can move them back to 1.14.1. -Jake
