Re: [VOTE] KIP-216: IQ should throw different exceptions for different errors

2020-01-22 Thread Vito Jeng
Thanks to everyone for voting. The KIP-216 has been accepted, with 3 binding votes from Bill, John and Matthias and 1 non-binding votes. --- Vito On Thu, Jan 23, 2020 at 1:46 PM Matthias J. Sax wrote: > +1 (binding) > > -Matthias > > On 1/17/20 9:35 PM, John Roesler wrote: > > Thanks for the

Re: [VOTE] KIP-216: IQ should throw different exceptions for different errors

2020-01-22 Thread Matthias J. Sax
+1 (binding) -Matthias On 1/17/20 9:35 PM, John Roesler wrote: > Thanks for the KIP! > > I'm +1 (binding) > > Thanks, > -John > > On Thu, Jan 16, 2020, at 08:46, Bill Bejeck wrote: >> Thanks for the KIP. >> >> +1 (binding) >> >> -Bill >> >> On Tue, Jan 14, 2020 at 9:41 AM Navinder Brar >> wro

Re: [VOTE] KIP-216: IQ should throw different exceptions for different errors

2020-01-17 Thread John Roesler
Thanks for the KIP! I'm +1 (binding) Thanks, -John On Thu, Jan 16, 2020, at 08:46, Bill Bejeck wrote: > Thanks for the KIP. > > +1 (binding) > > -Bill > > On Tue, Jan 14, 2020 at 9:41 AM Navinder Brar > wrote: > > > +1 (non-binding) With a small comment which was mentioned by Vinoth as > >

Re: [VOTE] KIP-216: IQ should throw different exceptions for different errors

2020-01-16 Thread Bill Bejeck
Thanks for the KIP. +1 (binding) -Bill On Tue, Jan 14, 2020 at 9:41 AM Navinder Brar wrote: > +1 (non-binding) With a small comment which was mentioned by Vinoth as > well. Did we fix on the flag for StreamsRebalancingException, I don't see > it in the KIP. > -Navinder > > > On Tuesday, 14

Re: [VOTE] KIP-216: IQ should throw different exceptions for different errors

2020-01-14 Thread Navinder Brar
+1 (non-binding) With a small comment which was mentioned by Vinoth as well. Did we fix on the flag for StreamsRebalancingException, I don't see it in the KIP. -Navinder On Tuesday, 14 January, 2020, 08:00:11 pm IST, Vito Jeng wrote: Hi, all, I would like to start the vote for KIP-2

[VOTE] KIP-216: IQ should throw different exceptions for different errors

2020-01-14 Thread Vito Jeng
Hi, all, I would like to start the vote for KIP-216. Currently, IQ throws InvalidStateStoreException for any types of error. With this KIP, user can distinguish different types of error. KIP is here: https://cwiki.apache.org/confluence/display/KAFKA/KIP-216%3A+IQ+should+throw+different+exception