Thanks everyone for reviewing and voting! I'm marking this KIP as accepted. There were 4 binding votes from Colin, Gwen, David and Bill, and 3 non-binding votes from Stanislav, M, and Mickael. There were no +0 or -1 votes.
Thanks again, Justine On Fri, Jul 19, 2019 at 9:10 AM Bill Bejeck <bbej...@gmail.com> wrote: > Thanks for the KIP, looks like a great addition. > > +1 (binding) > > -Bill > > On Fri, Jul 19, 2019 at 5:55 AM Mickael Maison <mickael.mai...@gmail.com> > wrote: > > > +1 (non binding) > > Thanks for the KIP! > > > > On Fri, Jul 19, 2019 at 2:23 AM David Arthur <davidart...@apache.org> > > wrote: > > > > > > +1 binding, looks like a nice improvement. Thanks! > > > > > > -David > > > > > > On Wed, Jul 17, 2019 at 6:17 PM Justine Olshan <jols...@confluent.io> > > wrote: > > > > > > > Hello all, > > > > > > > > I wanted to let you all know the KIP has been updated. The > > > > ComputedPartition class has been removed in favor of simply returning > > an > > > > integer to represent the record's partition. > > > > In short, the implications of this change mean that keyed records > will > > also > > > > trigger a change in the sticky partition. This was done for a case in > > which > > > > there may be keyed and non-keyed records. > > > > Upon testing, this did not significantly change the latency for > records > > > > with keyed values. > > > > > > > > Thank you, > > > > Justine > > > > > > > > On Sun, Jul 14, 2019 at 3:07 AM M. Manna <manme...@gmail.com> wrote: > > > > > > > > > +1(na) > > > > > > > > > > On Sat, 13 Jul 2019 at 22:17, Stanislav Kozlovski < > > > > stanis...@confluent.io> > > > > > wrote: > > > > > > > > > > > +1 (non-binding) > > > > > > > > > > > > Thanks! > > > > > > > > > > > > On Fri, Jul 12, 2019 at 6:02 PM Gwen Shapira <g...@confluent.io> > > > > wrote: > > > > > > > > > > > > > +1 (binding) > > > > > > > > > > > > > > Thank you for the KIP. This was long awaited. > > > > > > > > > > > > > > On Tue, Jul 9, 2019 at 5:15 PM Justine Olshan < > > jols...@confluent.io> > > > > > > > wrote: > > > > > > > > > > > > > > > > Hello all, > > > > > > > > > > > > > > > > I'd like to start the vote for KIP-480 : Sticky Partitioner. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-480%3A+Sticky+Partitioner > > > > > > > > > > > > > > > > Thank you, > > > > > > > > Justine Olshan > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > Gwen Shapira > > > > > > > Product Manager | Confluent > > > > > > > 650.450.2760 | @gwenshap > > > > > > > Follow us: Twitter | blog > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Best, > > > > > > Stanislav > > > > > > > > > > > > > > > > > >
