Vito, is there any update with regard to this KIP?
-Matthias On 11/5/17 6:11 PM, vito jeng wrote: > Thanks, Guozhang and Matthias. Your comments very useful for me. > > I'll update KIP and keep going on. > > > > --- > Vito > > On Sun, Nov 5, 2017 at 12:30 AM, Matthias J. Sax <matth...@confluent.io> > wrote: > >> Thanks for the KIP Vito! >> >> I agree with what Guozhang said. The original idea of the Jira was, to >> give different exceptions for different "recovery" strategies to the user. >> >> For example, if a store is currently recreated, a user just need to wait >> and can query the store later. On the other hand, if a store go migrated >> to another instance, a user needs to rediscover the store instead of a >> "plain retry". >> >> Fatal errors might be a third category. >> >> Not sure if there is something else? >> >> Anyway, the KIP should contain a section that talks about this ideas and >> reasoning. >> >> >> -Matthias >> >> >> On 11/3/17 11:26 PM, Guozhang Wang wrote: >>> Thanks for writing up the KIP. >>> >>> Vito, Matthias: one thing that I wanted to figure out first is what >>> categories of errors we want to notify the users, if we only wants to >>> distinguish fatal v.s. retriable then probably we should rename the >>> proposed StateStoreMigratedException / StateStoreClosedException classes. >>> And then from there we should list what are the possible internal >>> exceptions ever thrown in those APIs in the call trace, and which >>> exceptions should be wrapped to what others, and which ones should be >>> handled without re-throwing, and which ones should not be wrapped at all >>> but directly thrown to user's face. >>> >>> Guozhang >>> >>> >>> On Wed, Nov 1, 2017 at 11:09 PM, vito jeng <v...@is-land.com.tw> wrote: >>> >>>> Hi, >>>> >>>> I'd like to start discuss KIP-216: >>>> >>>> https://cwiki.apache.org/confluence/display/KAFKA/KIP- >>>> 216%3A+IQ+should+throw+different+exceptions+for+different+errors >>>> >>>> Please have a look. >>>> Thanks! >>>> >>>> --- >>>> Vito >>>> >>> >>> >>> >> >> >
signature.asc
Description: OpenPGP digital signature
