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 <wangg...@gmail.com> wrote: > 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 <mj...@apache.org> wrote: > > > +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. > > >> > > >> 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 output types are not `Void`, but > > due to > > >> type eraser we cannot distinguish the new overloaded function > signatures > > >> with the old ones if they also include `final String... > > stateStoreNames`. > > >> And in javadocs explains that if users want to connect state stores to > > this > > >> processor, they could use the `connectState` API instead. > > >> > > >> Otherwise, I'm +1. > > >> > > >> Guozhang > > >> > > >> On Tue, Feb 15, 2022 at 11:54 AM John Roesler <vvcep...@apache.org> > > wrote: > > >> > > >>> Thanks, Jorge! > > >>> > > >>> I'm +1 (binding) > > >>> > > >>> -John > > >>> > > >>> On Tue, 2022-02-15 at 19:16 +0000, 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 > > >>>> > > >>> > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-820%3A+Extend+KStream+process+with+new+Processor+API > > >>>> > > >>>> Thanks, > > >>>> Jorge > > >>> > > >>> > > >> > > >> -- > > >> -- Guozhang > > > > > -- > -- Guozhang >
