Re: [VOTE] KIP-820: Extend KStream process with new Processor API

2022-03-21 Thread Jorge Esteban Quilcate Otoya
Hi Dev team, The vote for KIP-820 has passed with: 3 +1 (binding) votes from John, Matthias, and Guozhang. This KIP is now accepted. Thanks for your feedback and votes! Jorge. On Thu, 17 Mar 2022 at 17:30, Guozhang Wang wrote: > Reviewed the updated docs, and recasting my +1 vote again,

Re: [VOTE] KIP-820: Extend KStream process with new Processor API

2022-03-17 Thread Guozhang Wang
Reviewed the updated docs, and recasting my +1 vote again, thanks Jorge for pushing it through! Guozhang On Tue, Mar 15, 2022 at 4:28 PM Matthias J. Sax wrote: > +1 (binding) > > Thanks for pushing this through. Was a difficult discussion! > > > -Matthias > > On 3/15/22 10:01 AM, John Roesler

Re: [VOTE] KIP-820: Extend KStream process with new Processor API

2022-03-15 Thread Matthias J. Sax
+1 (binding) Thanks for pushing this through. Was a difficult discussion! -Matthias On 3/15/22 10:01 AM, John Roesler wrote: Thanks for the update, Jorge! I’m still +1 (binding) Thanks, John On Thu, Feb 17, 2022, at 12:57, Guozhang Wang wrote: Thanks Jorge, overall looks good to me.

Re: [VOTE] KIP-820: Extend KStream process with new Processor API

2022-03-15 Thread John Roesler
Thanks for the update, Jorge! I’m still +1 (binding) Thanks, John On Thu, Feb 17, 2022, at 12:57, Guozhang Wang wrote: > Thanks Jorge, overall looks good to me. > > Maybe we can clarify a bit in the wiki that the reason we have to not > include the additional `final String... stateStoreNames`

Re: [VOTE] KIP-820: Extend KStream process with new Processor API

2022-02-17 Thread Guozhang Wang
Thanks Jorge, overall looks good to me. Maybe we can clarify a bit in the wiki that the reason we have to not include the additional `final String... stateStoreNames` params in the new `process` API is that we need to have overloaded functions which takes `ProcessorSupplier<...> ` where the

Re: [VOTE] KIP-820: Extend KStream process with new Processor API

2022-02-15 Thread John Roesler
Thanks, Jorge! I'm +1 (binding) -John On Tue, 2022-02-15 at 19:16 +, Jorge Esteban Quilcate Otoya wrote: > Hi all, > > I'd like to start a vote on KIP-820 which proposes extending KStream to use > the new Processor API >

[VOTE] KIP-820: Extend KStream process with new Processor API

2022-02-15 Thread Jorge Esteban Quilcate Otoya
Hi all, I'd like to start a vote on KIP-820 which proposes extending KStream to use the new Processor API https://cwiki.apache.org/confluence/display/KAFKA/KIP-820%3A+Extend+KStream+process+with+new+Processor+API Thanks, Jorge