+1 On Fri, Jul 21, 2017 at 12:08 PM, Guozhang Wang <wangg...@gmail.com> wrote:
> +1 > > On Thu, Jul 20, 2017 at 11:00 PM, Matthias J. Sax <matth...@confluent.io> > wrote: > > > +1 > > > > On 7/20/17 4:22 AM, Bill Bejeck wrote: > > > Hi, > > > > > > After working on the PR for this KIP I discovered that we need to add > and > > > additional parameter (TopicPartition) to the StateRestoreListener > > interface > > > methods. > > > > > > The addition of the TopicPartition is required as the > > StateRestoreListener > > > is for the entire application, thus all tasks with recovering state > > stores > > > call the same listener instance. The TopicPartition is needed to > > > disambiguate the progress of the state store recovery. > > > > > > For those that have voted before, please review the updated KIP > > > <https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > 167:+Add+interface+for+the+state+store+restoration+process> > > > and > > > re-vote. > > > > > > Thanks, > > > Bill > > > > > > > > > > -- > -- Guozhang >