Re: [Discuss] KIP-571: Add option to force remove members in StreamsResetter

2020-02-23 Thread Boyang Chen
tic+membership+protocol+to+reduce+consumer+rebalances> > for > details). > If it is used upon the older clusters like 2.3, > *UnsupportedVersionException > *will be thrown. > > Thanks! > Haoran > > ----------------------------

Re: [Discuss] KIP-571: Add option to force remove members in StreamsResetter

2020-02-18 Thread Boyang Chen
2020 at 9:29 PM feyman2009 > wrote: > > > > > Hi, Boyang > > > You can call me Feyman :) > > > Thanks for your quick reply with great advices! > > > I have updated the KIP-571 , would you mind to see if it looks > good ? > > > Thanks ! > > > > > >

Re: [Discuss] KIP-571: Add option to force remove members in StreamsResetter

2020-02-18 Thread Sophie Blee-Goldman
Thanks ! > > > > ---------------------------------- > > 发件人:Boyang Chen > > 发送时间:2020年2月14日(星期五) 08:35 > > 收件人:dev ; feyman2009 > > 主 题:Re: [Discuss] KIP-571: Add option to force remove members in > > StreamsResetter

Re: [Discuss] KIP-571: Add option to force remove members in StreamsResetter

2020-02-18 Thread Boyang Chen
at advices! > I have updated the KIP-571 , would you mind to see if it looks good ? > Thanks ! > > -- > 发件人:Boyang Chen > 发送时间:2020年2月14日(星期五) 08:35 > 收件人:dev ; feyman2009 > 主 题:Re: [Discuss] KIP-571:

Re: [Discuss] KIP-571: Add option to force remove members in StreamsResetter

2020-02-13 Thread Boyang Chen
Thanks for driving this change Feyman! Hope this is a good name to call you :) The motivation of the KIP looks good, and I have a couple of initial thoughts: 1. I guess the reason to use setters instead of adding a new constructor to MemberToRemove class is because we have two String members. Coul