Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

With 3 binding votes and 1 non-binding, the vote is closed and the KIP is accepted. I'm just wrapping up a first draft of a PR here: https://github.com/apache/kafka/pull/6824 Thanks! Paul On Thu, Sep 5, 2019 at 2:17 PM Paul Whalen wrote: > Thanks all! I updated the KIP status, and will get to

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

Thanks all! I updated the KIP status, and will get to the remaining TODOs in my PR hopefully this weekend. > On Sep 3, 2019, at 6:16 PM, Guozhang Wang wrote: > > Hi Paul, thanks for the confirmation! > > Since we have three binding votes now I think you can proceed and mark it > as accepted.

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

Hi Paul, thanks for the confirmation! Since we have three binding votes now I think you can proceed and mark it as accepted. On Tue, Sep 3, 2019 at 3:17 PM Paul Whalen wrote: > Yeah, agreed on it being the same reference. That’s the way I have it in > the working PR and I’ll update the KIP for

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

Yeah, agreed on it being the same reference. That’s the way I have it in the working PR and I’ll update the KIP for clarity. > On Sep 3, 2019, at 5:04 PM, Matthias J. Sax wrote: > > I am strongly in favor of "must be the same reference". > > > -Matthias > >> On 9/3/19 2:09 PM, Guozhang Wang

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

I am strongly in favor of "must be the same reference". -Matthias On 9/3/19 2:09 PM, Guozhang Wang wrote: > Hi Paul, > > Thanks for the KIP! +1 (binding). > > One minor comment about the following: > > "In order to solve the problem of addStateStore potentially being called > twice for the sa

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

Hi Paul, Thanks for the KIP! +1 (binding). One minor comment about the following: "In order to solve the problem of addStateStore potentially being called twice for the same store (because more than one Supplier specifies it), the check for duplicate stores in addStateStores will be relaxed to *

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

Thanks for the votes all! With two binding votes we’re in need of one more for the KIP to be accepted. With the 2.4 release coming in September, it would be great to get another committer to take a look soon so I could set aside some time to get implementation/documentation done to make it into

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

Thanks for the KIP. +1 (binding) On Tue, Aug 20, 2019 at 6:28 PM Matthias J. Sax wrote: > +1 (binding) > > > On 6/17/19 2:32 PM, John Roesler wrote: > > I'm +1 (nonbinding) on the current iteration of the proposal. > > > > On Mon, May 27, 2019 at 1:58 PM Paul Whalen wrote: > >> > >> I spoke to

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

+1 (binding) On 6/17/19 2:32 PM, John Roesler wrote: > I'm +1 (nonbinding) on the current iteration of the proposal. > > On Mon, May 27, 2019 at 1:58 PM Paul Whalen wrote: >> >> I spoke too early a month ago, but I believe the proposal is finalized now >> and ready for voting. >> >> KIP: >> htt

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

I'm +1 (nonbinding) on the current iteration of the proposal. On Mon, May 27, 2019 at 1:58 PM Paul Whalen wrote: > > I spoke too early a month ago, but I believe the proposal is finalized now > and ready for voting. > > KIP: > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=97553

Re: [VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

I spoke too early a month ago, but I believe the proposal is finalized now and ready for voting. KIP: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=97553756 Discussion: https://lists.apache.org/thread.html/600996d83d485f2b8daf45037de64a60cebdfac9b234bf3449b6b753@%3Cdev.kafka.ap

[VOTE] KIP-401: TransformerSupplier/ProcessorSupplier StateStore connecting

Hi all, After some good discussion on and adjustments to KIP-401 (which I renamed slightly for clarity), chatter has died down so I figured I may as well start a vote. KIP: TransformerSupplier/ProcessorSupplier StateStore connecting