+1 On Tue, 25 Jul 2017 at 18:17 Sriram Subramanian <r...@confluent.io> wrote:
> +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 > > >
