Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Thanks Guozhang,

For the PR review, the PR is already there in the KIP.

I am adding it here:

https://github.com/confluentinc/kafka/pull/242

It's not complete by any means but maybe one pass would be possible?

Thanks!
Sagar.

On Mon, Oct 5, 2020 at 10:26 PM Guozhang Wang  wrote:

> Hey Sagar, since the KIP is accepted, the next step would be to completing
> the PR review and finally merge it :)
>
> On Mon, Oct 5, 2020 at 9:46 AM Sagar  wrote:
>
> > Hi All,
> >
> > Just wanted to know, what the next steps are wrt to the KIP?
> >
> > Thanks!
> > Sagar.
> >
> >
> > On Thu, Sep 3, 2020 at 10:17 PM Sagar  wrote:
> >
> > > Hi John,
> > >
> > > Thank you! I have marked the KIP as Accepted :)
> > >
> > > Regarding the point on InMemoryKeyValueStore, in the PR I had added the
> > > implementation for InMemoryKeyValueStore as well. I hadn't mentioned
> > about
> > > it in the KIP which I have done now as you suggested.
> > >
> > > Thanks!
> > > Sagar.
> > >
> > > On Thu, Sep 3, 2020 at 8:10 PM John Roesler 
> wrote:
> > >
> > >> Hi Sagar,
> > >>
> > >> Yes! Congratulations :)
> > >>
> > >> Now, you can mark the status of the KIP as "Accepted" and we
> > >> can move on to reviewing your PRs.
> > >>
> > >> One quick note: Matthias didn't have time to review the KIP
> > >> in full, but he did point out to me that there's a lot of
> > >> information about the RocksDB implementation and no mention
> > >> of the InMemory store. We both agree that we should
> > >> implement the new method also for the InMemory store.
> > >> Assuming you agree, note that we don't need to discuss any
> > >> implementation details, so you could just update the KIP
> > >> document to also mention, "We will also implement the new
> > >> method in the InMemoryKeyValueStore."
> > >>
> > >> Thanks for your contribution to Apache Kafka!
> > >> -John
> > >>
> > >> On Thu, 2020-09-03 at 09:30 +0530, Sagar wrote:
> > >> > Thanks All!
> > >> >
> > >> > I see 3 binding +1 votes and 2 non-binding +1s. Does it mean this
> KIP
> > >> has
> > >> > gained a lazy majority?
> > >> >
> > >> > Thanks!
> > >> > Sagar.
> > >> >
> > >> > On Thu, Sep 3, 2020 at 6:51 AM Guozhang Wang 
> > >> wrote:
> > >> >
> > >> > > Thanks for the KIP Sagar. I'm +1 (binding) too.
> > >> > >
> > >> > >
> > >> > > Guozhang
> > >> > >
> > >> > > On Tue, Sep 1, 2020 at 1:24 PM Bill Bejeck 
> > wrote:
> > >> > >
> > >> > > > Thanks for the KIP! This is a great addition to the streams API.
> > >> > > >
> > >> > > > +1 (binding)
> > >> > > >
> > >> > > > -Bill
> > >> > > >
> > >> > > > On Tue, Sep 1, 2020 at 12:33 PM Sagar <
> sagarmeansoc...@gmail.com>
> > >> wrote:
> > >> > > >
> > >> > > > > Hi All,
> > >> > > > >
> > >> > > > > Bumping the thread again !
> > >> > > > >
> > >> > > > > Thanks!
> > >> > > > > Sagar.
> > >> > > > >
> > >> > > > > On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman <
> > >> > > sop...@confluent.io
> > >> > > > > wrote:
> > >> > > > >
> > >> > > > > > Thanks Sagar! +1 (non-binding)
> > >> > > > > >
> > >> > > > > > Sophie
> > >> > > > > >
> > >> > > > > > On Sun, Aug 2, 2020 at 11:37 PM Sagar <
> > >> sagarmeansoc...@gmail.com>
> > >> > > > wrote:
> > >> > > > > > > Hi All,
> > >> > > > > > >
> > >> > > > > > > Just thought of bumping this voting thread again to see if
> > we
> > >> can
> > >> > > > form
> > >> > > > > > any
> > >> > > > > > > consensus around this.
> > >> > > > > > >
> > >> > > > > > > Thanks!
> > >> > > > > > > Sagar.
> > >> > > > > > >
> > >> > > > > > >
> > >> > > > > > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
> > >> > > > > adam.bellem...@gmail.com
> > >> > > > > > > wrote:
> > >> > > > > > >
> > >> > > > > > > > LGTM
> > >> > > > > > > > +1 non-binding
> > >> > > > > > > >
> > >> > > > > > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar <
> > >> sagarmeansoc...@gmail.com
> > >> > > > > > wrote:
> > >> > > > > > > > > Hi All,
> > >> > > > > > > > >
> > >> > > > > > > > > Bumping this thread to see if there are any feedbacks.
> > >> > > > > > > > >
> > >> > > > > > > > > Thanks!
> > >> > > > > > > > > Sagar.
> > >> > > > > > > > >
> > >> > > > > > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler <
> > >> > > > vvcep...@apache.org>
> > >> > > > > > > > wrote:
> > >> > > > > > > > > > Thanks for the KIP, Sagar!
> > >> > > > > > > > > >
> > >> > > > > > > > > > I’m +1 (binding)
> > >> > > > > > > > > >
> > >> > > > > > > > > > -John
> > >> > > > > > > > > >
> > >> > > > > > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > >> > > > > > > > > > > Hi All,
> > >> > > > > > > > > > >
> > >> > > > > > > > > > > I would like to start a new voting thread for the
> > >> below KIP
> > >> > > > to
> > >> > > > > > add
> > >> > > > > > > > > prefix
> > >> > > > > > > > > > > scan support to state stores:
> > >> > > > > > > > > > >
> > >> > > > > > > > > > >
> > >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > >> > > > > > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > >> > > > > > > > > > > <
> > >> > >
> 

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Guozhang Wang]

Hey Sagar, since the KIP is accepted, the next step would be to completing
the PR review and finally merge it :)

On Mon, Oct 5, 2020 at 9:46 AM Sagar  wrote:

> Hi All,
>
> Just wanted to know, what the next steps are wrt to the KIP?
>
> Thanks!
> Sagar.
>
>
> On Thu, Sep 3, 2020 at 10:17 PM Sagar  wrote:
>
> > Hi John,
> >
> > Thank you! I have marked the KIP as Accepted :)
> >
> > Regarding the point on InMemoryKeyValueStore, in the PR I had added the
> > implementation for InMemoryKeyValueStore as well. I hadn't mentioned
> about
> > it in the KIP which I have done now as you suggested.
> >
> > Thanks!
> > Sagar.
> >
> > On Thu, Sep 3, 2020 at 8:10 PM John Roesler  wrote:
> >
> >> Hi Sagar,
> >>
> >> Yes! Congratulations :)
> >>
> >> Now, you can mark the status of the KIP as "Accepted" and we
> >> can move on to reviewing your PRs.
> >>
> >> One quick note: Matthias didn't have time to review the KIP
> >> in full, but he did point out to me that there's a lot of
> >> information about the RocksDB implementation and no mention
> >> of the InMemory store. We both agree that we should
> >> implement the new method also for the InMemory store.
> >> Assuming you agree, note that we don't need to discuss any
> >> implementation details, so you could just update the KIP
> >> document to also mention, "We will also implement the new
> >> method in the InMemoryKeyValueStore."
> >>
> >> Thanks for your contribution to Apache Kafka!
> >> -John
> >>
> >> On Thu, 2020-09-03 at 09:30 +0530, Sagar wrote:
> >> > Thanks All!
> >> >
> >> > I see 3 binding +1 votes and 2 non-binding +1s. Does it mean this KIP
> >> has
> >> > gained a lazy majority?
> >> >
> >> > Thanks!
> >> > Sagar.
> >> >
> >> > On Thu, Sep 3, 2020 at 6:51 AM Guozhang Wang 
> >> wrote:
> >> >
> >> > > Thanks for the KIP Sagar. I'm +1 (binding) too.
> >> > >
> >> > >
> >> > > Guozhang
> >> > >
> >> > > On Tue, Sep 1, 2020 at 1:24 PM Bill Bejeck 
> wrote:
> >> > >
> >> > > > Thanks for the KIP! This is a great addition to the streams API.
> >> > > >
> >> > > > +1 (binding)
> >> > > >
> >> > > > -Bill
> >> > > >
> >> > > > On Tue, Sep 1, 2020 at 12:33 PM Sagar 
> >> wrote:
> >> > > >
> >> > > > > Hi All,
> >> > > > >
> >> > > > > Bumping the thread again !
> >> > > > >
> >> > > > > Thanks!
> >> > > > > Sagar.
> >> > > > >
> >> > > > > On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman <
> >> > > sop...@confluent.io
> >> > > > > wrote:
> >> > > > >
> >> > > > > > Thanks Sagar! +1 (non-binding)
> >> > > > > >
> >> > > > > > Sophie
> >> > > > > >
> >> > > > > > On Sun, Aug 2, 2020 at 11:37 PM Sagar <
> >> sagarmeansoc...@gmail.com>
> >> > > > wrote:
> >> > > > > > > Hi All,
> >> > > > > > >
> >> > > > > > > Just thought of bumping this voting thread again to see if
> we
> >> can
> >> > > > form
> >> > > > > > any
> >> > > > > > > consensus around this.
> >> > > > > > >
> >> > > > > > > Thanks!
> >> > > > > > > Sagar.
> >> > > > > > >
> >> > > > > > >
> >> > > > > > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
> >> > > > > adam.bellem...@gmail.com
> >> > > > > > > wrote:
> >> > > > > > >
> >> > > > > > > > LGTM
> >> > > > > > > > +1 non-binding
> >> > > > > > > >
> >> > > > > > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar <
> >> sagarmeansoc...@gmail.com
> >> > > > > > wrote:
> >> > > > > > > > > Hi All,
> >> > > > > > > > >
> >> > > > > > > > > Bumping this thread to see if there are any feedbacks.
> >> > > > > > > > >
> >> > > > > > > > > Thanks!
> >> > > > > > > > > Sagar.
> >> > > > > > > > >
> >> > > > > > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler <
> >> > > > vvcep...@apache.org>
> >> > > > > > > > wrote:
> >> > > > > > > > > > Thanks for the KIP, Sagar!
> >> > > > > > > > > >
> >> > > > > > > > > > I’m +1 (binding)
> >> > > > > > > > > >
> >> > > > > > > > > > -John
> >> > > > > > > > > >
> >> > > > > > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> >> > > > > > > > > > > Hi All,
> >> > > > > > > > > > >
> >> > > > > > > > > > > I would like to start a new voting thread for the
> >> below KIP
> >> > > > to
> >> > > > > > add
> >> > > > > > > > > prefix
> >> > > > > > > > > > > scan support to state stores:
> >> > > > > > > > > > >
> >> > > > > > > > > > >
> >> https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> >> > > > > > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> >> > > > > > > > > > > <
> >> > >
> >>
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> >> > > > > > > > > > >
> >> > > > > > > > > > > Thanks!
> >> > > > > > > > > > > Sagar.
> >> > > > > > > > > > >
> >> > >
> >> > > --
> >> > > -- Guozhang
> >> > >
> >>
> >>
>


-- 
-- Guozhang

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Hi All,

Just wanted to know, what the next steps are wrt to the KIP?

Thanks!
Sagar.


On Thu, Sep 3, 2020 at 10:17 PM Sagar  wrote:

> Hi John,
>
> Thank you! I have marked the KIP as Accepted :)
>
> Regarding the point on InMemoryKeyValueStore, in the PR I had added the
> implementation for InMemoryKeyValueStore as well. I hadn't mentioned about
> it in the KIP which I have done now as you suggested.
>
> Thanks!
> Sagar.
>
> On Thu, Sep 3, 2020 at 8:10 PM John Roesler  wrote:
>
>> Hi Sagar,
>>
>> Yes! Congratulations :)
>>
>> Now, you can mark the status of the KIP as "Accepted" and we
>> can move on to reviewing your PRs.
>>
>> One quick note: Matthias didn't have time to review the KIP
>> in full, but he did point out to me that there's a lot of
>> information about the RocksDB implementation and no mention
>> of the InMemory store. We both agree that we should
>> implement the new method also for the InMemory store.
>> Assuming you agree, note that we don't need to discuss any
>> implementation details, so you could just update the KIP
>> document to also mention, "We will also implement the new
>> method in the InMemoryKeyValueStore."
>>
>> Thanks for your contribution to Apache Kafka!
>> -John
>>
>> On Thu, 2020-09-03 at 09:30 +0530, Sagar wrote:
>> > Thanks All!
>> >
>> > I see 3 binding +1 votes and 2 non-binding +1s. Does it mean this KIP
>> has
>> > gained a lazy majority?
>> >
>> > Thanks!
>> > Sagar.
>> >
>> > On Thu, Sep 3, 2020 at 6:51 AM Guozhang Wang 
>> wrote:
>> >
>> > > Thanks for the KIP Sagar. I'm +1 (binding) too.
>> > >
>> > >
>> > > Guozhang
>> > >
>> > > On Tue, Sep 1, 2020 at 1:24 PM Bill Bejeck  wrote:
>> > >
>> > > > Thanks for the KIP! This is a great addition to the streams API.
>> > > >
>> > > > +1 (binding)
>> > > >
>> > > > -Bill
>> > > >
>> > > > On Tue, Sep 1, 2020 at 12:33 PM Sagar 
>> wrote:
>> > > >
>> > > > > Hi All,
>> > > > >
>> > > > > Bumping the thread again !
>> > > > >
>> > > > > Thanks!
>> > > > > Sagar.
>> > > > >
>> > > > > On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman <
>> > > sop...@confluent.io
>> > > > > wrote:
>> > > > >
>> > > > > > Thanks Sagar! +1 (non-binding)
>> > > > > >
>> > > > > > Sophie
>> > > > > >
>> > > > > > On Sun, Aug 2, 2020 at 11:37 PM Sagar <
>> sagarmeansoc...@gmail.com>
>> > > > wrote:
>> > > > > > > Hi All,
>> > > > > > >
>> > > > > > > Just thought of bumping this voting thread again to see if we
>> can
>> > > > form
>> > > > > > any
>> > > > > > > consensus around this.
>> > > > > > >
>> > > > > > > Thanks!
>> > > > > > > Sagar.
>> > > > > > >
>> > > > > > >
>> > > > > > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
>> > > > > adam.bellem...@gmail.com
>> > > > > > > wrote:
>> > > > > > >
>> > > > > > > > LGTM
>> > > > > > > > +1 non-binding
>> > > > > > > >
>> > > > > > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar <
>> sagarmeansoc...@gmail.com
>> > > > > > wrote:
>> > > > > > > > > Hi All,
>> > > > > > > > >
>> > > > > > > > > Bumping this thread to see if there are any feedbacks.
>> > > > > > > > >
>> > > > > > > > > Thanks!
>> > > > > > > > > Sagar.
>> > > > > > > > >
>> > > > > > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler <
>> > > > vvcep...@apache.org>
>> > > > > > > > wrote:
>> > > > > > > > > > Thanks for the KIP, Sagar!
>> > > > > > > > > >
>> > > > > > > > > > I’m +1 (binding)
>> > > > > > > > > >
>> > > > > > > > > > -John
>> > > > > > > > > >
>> > > > > > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
>> > > > > > > > > > > Hi All,
>> > > > > > > > > > >
>> > > > > > > > > > > I would like to start a new voting thread for the
>> below KIP
>> > > > to
>> > > > > > add
>> > > > > > > > > prefix
>> > > > > > > > > > > scan support to state stores:
>> > > > > > > > > > >
>> > > > > > > > > > >
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-
>> > > > > > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
>> > > > > > > > > > > <
>> > >
>> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
>> > > > > > > > > > >
>> > > > > > > > > > > Thanks!
>> > > > > > > > > > > Sagar.
>> > > > > > > > > > >
>> > >
>> > > --
>> > > -- Guozhang
>> > >
>>
>>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Hi John,

Thank you! I have marked the KIP as Accepted :)

Regarding the point on InMemoryKeyValueStore, in the PR I had added the
implementation for InMemoryKeyValueStore as well. I hadn't mentioned about
it in the KIP which I have done now as you suggested.

Thanks!
Sagar.

On Thu, Sep 3, 2020 at 8:10 PM John Roesler  wrote:

> Hi Sagar,
>
> Yes! Congratulations :)
>
> Now, you can mark the status of the KIP as "Accepted" and we
> can move on to reviewing your PRs.
>
> One quick note: Matthias didn't have time to review the KIP
> in full, but he did point out to me that there's a lot of
> information about the RocksDB implementation and no mention
> of the InMemory store. We both agree that we should
> implement the new method also for the InMemory store.
> Assuming you agree, note that we don't need to discuss any
> implementation details, so you could just update the KIP
> document to also mention, "We will also implement the new
> method in the InMemoryKeyValueStore."
>
> Thanks for your contribution to Apache Kafka!
> -John
>
> On Thu, 2020-09-03 at 09:30 +0530, Sagar wrote:
> > Thanks All!
> >
> > I see 3 binding +1 votes and 2 non-binding +1s. Does it mean this KIP has
> > gained a lazy majority?
> >
> > Thanks!
> > Sagar.
> >
> > On Thu, Sep 3, 2020 at 6:51 AM Guozhang Wang  wrote:
> >
> > > Thanks for the KIP Sagar. I'm +1 (binding) too.
> > >
> > >
> > > Guozhang
> > >
> > > On Tue, Sep 1, 2020 at 1:24 PM Bill Bejeck  wrote:
> > >
> > > > Thanks for the KIP! This is a great addition to the streams API.
> > > >
> > > > +1 (binding)
> > > >
> > > > -Bill
> > > >
> > > > On Tue, Sep 1, 2020 at 12:33 PM Sagar 
> wrote:
> > > >
> > > > > Hi All,
> > > > >
> > > > > Bumping the thread again !
> > > > >
> > > > > Thanks!
> > > > > Sagar.
> > > > >
> > > > > On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman <
> > > sop...@confluent.io
> > > > > wrote:
> > > > >
> > > > > > Thanks Sagar! +1 (non-binding)
> > > > > >
> > > > > > Sophie
> > > > > >
> > > > > > On Sun, Aug 2, 2020 at 11:37 PM Sagar  >
> > > > wrote:
> > > > > > > Hi All,
> > > > > > >
> > > > > > > Just thought of bumping this voting thread again to see if we
> can
> > > > form
> > > > > > any
> > > > > > > consensus around this.
> > > > > > >
> > > > > > > Thanks!
> > > > > > > Sagar.
> > > > > > >
> > > > > > >
> > > > > > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
> > > > > adam.bellem...@gmail.com
> > > > > > > wrote:
> > > > > > >
> > > > > > > > LGTM
> > > > > > > > +1 non-binding
> > > > > > > >
> > > > > > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar <
> sagarmeansoc...@gmail.com
> > > > > > wrote:
> > > > > > > > > Hi All,
> > > > > > > > >
> > > > > > > > > Bumping this thread to see if there are any feedbacks.
> > > > > > > > >
> > > > > > > > > Thanks!
> > > > > > > > > Sagar.
> > > > > > > > >
> > > > > > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler <
> > > > vvcep...@apache.org>
> > > > > > > > wrote:
> > > > > > > > > > Thanks for the KIP, Sagar!
> > > > > > > > > >
> > > > > > > > > > I’m +1 (binding)
> > > > > > > > > >
> > > > > > > > > > -John
> > > > > > > > > >
> > > > > > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > > > > > > > > Hi All,
> > > > > > > > > > >
> > > > > > > > > > > I would like to start a new voting thread for the
> below KIP
> > > > to
> > > > > > add
> > > > > > > > > prefix
> > > > > > > > > > > scan support to state stores:
> > > > > > > > > > >
> > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > > > > > <
> > >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > > > > >
> > > > > > > > > > > Thanks!
> > > > > > > > > > > Sagar.
> > > > > > > > > > >
> > >
> > > --
> > > -- Guozhang
> > >
>
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[John Roesler]

Hi Sagar,

Yes! Congratulations :)

Now, you can mark the status of the KIP as "Accepted" and we
can move on to reviewing your PRs.

One quick note: Matthias didn't have time to review the KIP
in full, but he did point out to me that there's a lot of
information about the RocksDB implementation and no mention
of the InMemory store. We both agree that we should
implement the new method also for the InMemory store.
Assuming you agree, note that we don't need to discuss any
implementation details, so you could just update the KIP
document to also mention, "We will also implement the new
method in the InMemoryKeyValueStore."

Thanks for your contribution to Apache Kafka!
-John

On Thu, 2020-09-03 at 09:30 +0530, Sagar wrote:
> Thanks All!
> 
> I see 3 binding +1 votes and 2 non-binding +1s. Does it mean this KIP has
> gained a lazy majority?
> 
> Thanks!
> Sagar.
> 
> On Thu, Sep 3, 2020 at 6:51 AM Guozhang Wang  wrote:
> 
> > Thanks for the KIP Sagar. I'm +1 (binding) too.
> > 
> > 
> > Guozhang
> > 
> > On Tue, Sep 1, 2020 at 1:24 PM Bill Bejeck  wrote:
> > 
> > > Thanks for the KIP! This is a great addition to the streams API.
> > > 
> > > +1 (binding)
> > > 
> > > -Bill
> > > 
> > > On Tue, Sep 1, 2020 at 12:33 PM Sagar  wrote:
> > > 
> > > > Hi All,
> > > > 
> > > > Bumping the thread again !
> > > > 
> > > > Thanks!
> > > > Sagar.
> > > > 
> > > > On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman <
> > sop...@confluent.io
> > > > wrote:
> > > > 
> > > > > Thanks Sagar! +1 (non-binding)
> > > > > 
> > > > > Sophie
> > > > > 
> > > > > On Sun, Aug 2, 2020 at 11:37 PM Sagar 
> > > wrote:
> > > > > > Hi All,
> > > > > > 
> > > > > > Just thought of bumping this voting thread again to see if we can
> > > form
> > > > > any
> > > > > > consensus around this.
> > > > > > 
> > > > > > Thanks!
> > > > > > Sagar.
> > > > > > 
> > > > > > 
> > > > > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
> > > > adam.bellem...@gmail.com
> > > > > > wrote:
> > > > > > 
> > > > > > > LGTM
> > > > > > > +1 non-binding
> > > > > > > 
> > > > > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar  > > > > wrote:
> > > > > > > > Hi All,
> > > > > > > > 
> > > > > > > > Bumping this thread to see if there are any feedbacks.
> > > > > > > > 
> > > > > > > > Thanks!
> > > > > > > > Sagar.
> > > > > > > > 
> > > > > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler <
> > > vvcep...@apache.org>
> > > > > > > wrote:
> > > > > > > > > Thanks for the KIP, Sagar!
> > > > > > > > > 
> > > > > > > > > I’m +1 (binding)
> > > > > > > > > 
> > > > > > > > > -John
> > > > > > > > > 
> > > > > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > > > > > > > Hi All,
> > > > > > > > > > 
> > > > > > > > > > I would like to start a new voting thread for the below KIP
> > > to
> > > > > add
> > > > > > > > prefix
> > > > > > > > > > scan support to state stores:
> > > > > > > > > > 
> > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > > > > <
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > > > > 
> > > > > > > > > > Thanks!
> > > > > > > > > > Sagar.
> > > > > > > > > > 
> > 
> > --
> > -- Guozhang
> > 

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Thanks All!

I see 3 binding +1 votes and 2 non-binding +1s. Does it mean this KIP has
gained a lazy majority?

Thanks!
Sagar.

On Thu, Sep 3, 2020 at 6:51 AM Guozhang Wang  wrote:

> Thanks for the KIP Sagar. I'm +1 (binding) too.
>
>
> Guozhang
>
> On Tue, Sep 1, 2020 at 1:24 PM Bill Bejeck  wrote:
>
> > Thanks for the KIP! This is a great addition to the streams API.
> >
> > +1 (binding)
> >
> > -Bill
> >
> > On Tue, Sep 1, 2020 at 12:33 PM Sagar  wrote:
> >
> > > Hi All,
> > >
> > > Bumping the thread again !
> > >
> > > Thanks!
> > > Sagar.
> > >
> > > On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman <
> sop...@confluent.io
> > >
> > > wrote:
> > >
> > > > Thanks Sagar! +1 (non-binding)
> > > >
> > > > Sophie
> > > >
> > > > On Sun, Aug 2, 2020 at 11:37 PM Sagar 
> > wrote:
> > > >
> > > > > Hi All,
> > > > >
> > > > > Just thought of bumping this voting thread again to see if we can
> > form
> > > > any
> > > > > consensus around this.
> > > > >
> > > > > Thanks!
> > > > > Sagar.
> > > > >
> > > > >
> > > > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
> > > adam.bellem...@gmail.com
> > > > >
> > > > > wrote:
> > > > >
> > > > > > LGTM
> > > > > > +1 non-binding
> > > > > >
> > > > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar  >
> > > > wrote:
> > > > > >
> > > > > > > Hi All,
> > > > > > >
> > > > > > > Bumping this thread to see if there are any feedbacks.
> > > > > > >
> > > > > > > Thanks!
> > > > > > > Sagar.
> > > > > > >
> > > > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler <
> > vvcep...@apache.org>
> > > > > > wrote:
> > > > > > >
> > > > > > > > Thanks for the KIP, Sagar!
> > > > > > > >
> > > > > > > > I’m +1 (binding)
> > > > > > > >
> > > > > > > > -John
> > > > > > > >
> > > > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > > > > > > Hi All,
> > > > > > > > >
> > > > > > > > > I would like to start a new voting thread for the below KIP
> > to
> > > > add
> > > > > > > prefix
> > > > > > > > > scan support to state stores:
> > > > > > > > >
> > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > > > <
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > Thanks!
> > > > > > > > > Sagar.
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>
>
> --
> -- Guozhang
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Guozhang Wang]

Thanks for the KIP Sagar. I'm +1 (binding) too.


Guozhang

On Tue, Sep 1, 2020 at 1:24 PM Bill Bejeck  wrote:

> Thanks for the KIP! This is a great addition to the streams API.
>
> +1 (binding)
>
> -Bill
>
> On Tue, Sep 1, 2020 at 12:33 PM Sagar  wrote:
>
> > Hi All,
> >
> > Bumping the thread again !
> >
> > Thanks!
> > Sagar.
> >
> > On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman  >
> > wrote:
> >
> > > Thanks Sagar! +1 (non-binding)
> > >
> > > Sophie
> > >
> > > On Sun, Aug 2, 2020 at 11:37 PM Sagar 
> wrote:
> > >
> > > > Hi All,
> > > >
> > > > Just thought of bumping this voting thread again to see if we can
> form
> > > any
> > > > consensus around this.
> > > >
> > > > Thanks!
> > > > Sagar.
> > > >
> > > >
> > > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
> > adam.bellem...@gmail.com
> > > >
> > > > wrote:
> > > >
> > > > > LGTM
> > > > > +1 non-binding
> > > > >
> > > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar 
> > > wrote:
> > > > >
> > > > > > Hi All,
> > > > > >
> > > > > > Bumping this thread to see if there are any feedbacks.
> > > > > >
> > > > > > Thanks!
> > > > > > Sagar.
> > > > > >
> > > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler <
> vvcep...@apache.org>
> > > > > wrote:
> > > > > >
> > > > > > > Thanks for the KIP, Sagar!
> > > > > > >
> > > > > > > I’m +1 (binding)
> > > > > > >
> > > > > > > -John
> > > > > > >
> > > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > > > > > Hi All,
> > > > > > > >
> > > > > > > > I would like to start a new voting thread for the below KIP
> to
> > > add
> > > > > > prefix
> > > > > > > > scan support to state stores:
> > > > > > > >
> > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > > <
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > >
> > > > > > > >
> > > > > > > > Thanks!
> > > > > > > > Sagar.
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>


-- 
-- Guozhang

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Bill Bejeck]

Thanks for the KIP! This is a great addition to the streams API.

+1 (binding)

-Bill

On Tue, Sep 1, 2020 at 12:33 PM Sagar  wrote:

> Hi All,
>
> Bumping the thread again !
>
> Thanks!
> Sagar.
>
> On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman 
> wrote:
>
> > Thanks Sagar! +1 (non-binding)
> >
> > Sophie
> >
> > On Sun, Aug 2, 2020 at 11:37 PM Sagar  wrote:
> >
> > > Hi All,
> > >
> > > Just thought of bumping this voting thread again to see if we can form
> > any
> > > consensus around this.
> > >
> > > Thanks!
> > > Sagar.
> > >
> > >
> > > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare <
> adam.bellem...@gmail.com
> > >
> > > wrote:
> > >
> > > > LGTM
> > > > +1 non-binding
> > > >
> > > > On Sun, Jul 19, 2020 at 4:13 AM Sagar 
> > wrote:
> > > >
> > > > > Hi All,
> > > > >
> > > > > Bumping this thread to see if there are any feedbacks.
> > > > >
> > > > > Thanks!
> > > > > Sagar.
> > > > >
> > > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler 
> > > > wrote:
> > > > >
> > > > > > Thanks for the KIP, Sagar!
> > > > > >
> > > > > > I’m +1 (binding)
> > > > > >
> > > > > > -John
> > > > > >
> > > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > > > > Hi All,
> > > > > > >
> > > > > > > I would like to start a new voting thread for the below KIP to
> > add
> > > > > prefix
> > > > > > > scan support to state stores:
> > > > > > >
> > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > > <
> > > > > >
> > > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > >
> > > > > > >
> > > > > > > Thanks!
> > > > > > > Sagar.
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Hi All,

Bumping the thread again !

Thanks!
Sagar.

On Wed, Aug 5, 2020 at 12:08 AM Sophie Blee-Goldman 
wrote:

> Thanks Sagar! +1 (non-binding)
>
> Sophie
>
> On Sun, Aug 2, 2020 at 11:37 PM Sagar  wrote:
>
> > Hi All,
> >
> > Just thought of bumping this voting thread again to see if we can form
> any
> > consensus around this.
> >
> > Thanks!
> > Sagar.
> >
> >
> > On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare  >
> > wrote:
> >
> > > LGTM
> > > +1 non-binding
> > >
> > > On Sun, Jul 19, 2020 at 4:13 AM Sagar 
> wrote:
> > >
> > > > Hi All,
> > > >
> > > > Bumping this thread to see if there are any feedbacks.
> > > >
> > > > Thanks!
> > > > Sagar.
> > > >
> > > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler 
> > > wrote:
> > > >
> > > > > Thanks for the KIP, Sagar!
> > > > >
> > > > > I’m +1 (binding)
> > > > >
> > > > > -John
> > > > >
> > > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > > > Hi All,
> > > > > >
> > > > > > I would like to start a new voting thread for the below KIP to
> add
> > > > prefix
> > > > > > scan support to state stores:
> > > > > >
> > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > > <
> > > > >
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > >
> > > > > >
> > > > > > Thanks!
> > > > > > Sagar.
> > > > > >
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sophie Blee-Goldman]

Thanks Sagar! +1 (non-binding)

Sophie

On Sun, Aug 2, 2020 at 11:37 PM Sagar  wrote:

> Hi All,
>
> Just thought of bumping this voting thread again to see if we can form any
> consensus around this.
>
> Thanks!
> Sagar.
>
>
> On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare 
> wrote:
>
> > LGTM
> > +1 non-binding
> >
> > On Sun, Jul 19, 2020 at 4:13 AM Sagar  wrote:
> >
> > > Hi All,
> > >
> > > Bumping this thread to see if there are any feedbacks.
> > >
> > > Thanks!
> > > Sagar.
> > >
> > > On Tue, Jul 14, 2020 at 9:49 AM John Roesler 
> > wrote:
> > >
> > > > Thanks for the KIP, Sagar!
> > > >
> > > > I’m +1 (binding)
> > > >
> > > > -John
> > > >
> > > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > > Hi All,
> > > > >
> > > > > I would like to start a new voting thread for the below KIP to add
> > > prefix
> > > > > scan support to state stores:
> > > > >
> > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > > <
> > > >
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > >
> > > > >
> > > > > Thanks!
> > > > > Sagar.
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Hi All,

Just thought of bumping this voting thread again to see if we can form any
consensus around this.

Thanks!
Sagar.


On Mon, Jul 20, 2020 at 4:21 AM Adam Bellemare 
wrote:

> LGTM
> +1 non-binding
>
> On Sun, Jul 19, 2020 at 4:13 AM Sagar  wrote:
>
> > Hi All,
> >
> > Bumping this thread to see if there are any feedbacks.
> >
> > Thanks!
> > Sagar.
> >
> > On Tue, Jul 14, 2020 at 9:49 AM John Roesler 
> wrote:
> >
> > > Thanks for the KIP, Sagar!
> > >
> > > I’m +1 (binding)
> > >
> > > -John
> > >
> > > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > > Hi All,
> > > >
> > > > I would like to start a new voting thread for the below KIP to add
> > prefix
> > > > scan support to state stores:
> > > >
> > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > > <
> > >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > >
> > > >
> > > > Thanks!
> > > > Sagar.
> > > >
> > >
> >
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Adam Bellemare]

LGTM
+1 non-binding

On Sun, Jul 19, 2020 at 4:13 AM Sagar  wrote:

> Hi All,
>
> Bumping this thread to see if there are any feedbacks.
>
> Thanks!
> Sagar.
>
> On Tue, Jul 14, 2020 at 9:49 AM John Roesler  wrote:
>
> > Thanks for the KIP, Sagar!
> >
> > I’m +1 (binding)
> >
> > -John
> >
> > On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > > Hi All,
> > >
> > > I would like to start a new voting thread for the below KIP to add
> prefix
> > > scan support to state stores:
> > >
> > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > > <
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> > >
> > >
> > > Thanks!
> > > Sagar.
> > >
> >
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Hi All,

Bumping this thread to see if there are any feedbacks.

Thanks!
Sagar.

On Tue, Jul 14, 2020 at 9:49 AM John Roesler  wrote:

> Thanks for the KIP, Sagar!
>
> I’m +1 (binding)
>
> -John
>
> On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> > Hi All,
> >
> > I would like to start a new voting thread for the below KIP to add prefix
> > scan support to state stores:
> >
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > 614%3A+Add+Prefix+Scan+support+for+State+Stores
> > <
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-614%3A+Add+Prefix+Scan+support+for+State+Stores
> >
> >
> > Thanks!
> > Sagar.
> >
>

Re: [VOTE] KIP-614: Add Prefix Scan support for State Stores

[John Roesler]

Thanks for the KIP, Sagar!

I’m +1 (binding)

-John

On Sun, Jul 12, 2020, at 02:05, Sagar wrote:
> Hi All,
> 
> I would like to start a new voting thread for the below KIP to add prefix
> scan support to state stores:
> 
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> 614%3A+Add+Prefix+Scan+support+for+State+Stores
> 
> 
> Thanks!
> Sagar.
>

[VOTE] KIP-614: Add Prefix Scan support for State Stores

[Sagar]

Hi All,

I would like to start a new voting thread for the below KIP to add prefix
scan support to state stores:

https://cwiki.apache.org/confluence/display/KAFKA/KIP-
614%3A+Add+Prefix+Scan+support+for+State+Stores


Thanks!
Sagar.