+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 > > > > > >
