You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by Richard Yu <yo...@gmail.com> on 2017/10/24 03:11:17 UTC
[VOTE] KIP-205: Add all() and range() API to ReadOnlyWindowStore
Hi all,
I want to propose KIP-205 for the addition of new API. It is about adding
methods similar to those found in ReadOnlyKeyValueStore to the
ReadOnlyWindowStore class. As it appears the discussion has reached a
conclusion, I would like to start the voting process.
https://cwiki.apache.org/confluence/display/KAFKA/KIP-205%3A+Add+all%28%29+and+range%28%29+API+to+ReadOnlyWindowStore
Thanks for your patience!
Re: [VOTE] KIP-205: Add all() and range() API to ReadOnlyWindowStore
Posted by Guozhang Wang <wa...@gmail.com>.
Since we already have three binding +1s for this KIP, I'm closing this
thread as accepted and updating the KIP wiki accordingly.
Guozhang
On Mon, Oct 30, 2017 at 2:10 PM, Gwen Shapira <gw...@confluent.io> wrote:
> +1 for the KIP and Rename
>
> On Sat, Oct 28, 2017 at 5:51 AM Bill Bejeck <bb...@gmail.com> wrote:
>
> > +1 for the KIP and +1 on renaming "range" to "fetchAll"
> >
> > Thanks,
> > Bill
> >
> > On Thu, Oct 26, 2017 at 4:39 PM, Guozhang Wang <wa...@gmail.com>
> wrote:
> >
> > > BTW, Xavier has an argument to rename "range" as it does not take a key
> > or
> > > key range at all. I'd propose we rename this function to "fetchAll".
> > >
> > > Guozhang
> > >
> > > On Wed, Oct 25, 2017 at 7:27 AM, Damian Guy <da...@gmail.com>
> > wrote:
> > >
> > > > +1
> > > >
> > > > On Tue, 24 Oct 2017 at 16:46 Guozhang Wang <wa...@gmail.com>
> wrote:
> > > >
> > > > > +1. Thanks.
> > > > >
> > > > > On Mon, Oct 23, 2017 at 8:11 PM, Richard Yu <
> > > yohan.richard.yu@gmail.com>
> > > > > wrote:
> > > > >
> > > > > > Hi all,
> > > > > >
> > > > > > I want to propose KIP-205 for the addition of new API. It is
> about
> > > > adding
> > > > > > methods similar to those found in ReadOnlyKeyValueStore to the
> > > > > > ReadOnlyWindowStore class. As it appears the discussion has
> > reached a
> > > > > > conclusion, I would like to start the voting process.
> > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > > 205%3A+Add+all%28%29+and+range%28%29+API+to+ReadOnlyWindowStore
> > > > > >
> > > > > > Thanks for your patience!
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > -- Guozhang
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > -- Guozhang
> > >
> >
>
--
-- Guozhang
Re: [VOTE] KIP-205: Add all() and range() API to ReadOnlyWindowStore
Posted by Gwen Shapira <gw...@confluent.io>.
+1 for the KIP and Rename
On Sat, Oct 28, 2017 at 5:51 AM Bill Bejeck <bb...@gmail.com> wrote:
> +1 for the KIP and +1 on renaming "range" to "fetchAll"
>
> Thanks,
> Bill
>
> On Thu, Oct 26, 2017 at 4:39 PM, Guozhang Wang <wa...@gmail.com> wrote:
>
> > BTW, Xavier has an argument to rename "range" as it does not take a key
> or
> > key range at all. I'd propose we rename this function to "fetchAll".
> >
> > Guozhang
> >
> > On Wed, Oct 25, 2017 at 7:27 AM, Damian Guy <da...@gmail.com>
> wrote:
> >
> > > +1
> > >
> > > On Tue, 24 Oct 2017 at 16:46 Guozhang Wang <wa...@gmail.com> wrote:
> > >
> > > > +1. Thanks.
> > > >
> > > > On Mon, Oct 23, 2017 at 8:11 PM, Richard Yu <
> > yohan.richard.yu@gmail.com>
> > > > wrote:
> > > >
> > > > > Hi all,
> > > > >
> > > > > I want to propose KIP-205 for the addition of new API. It is about
> > > adding
> > > > > methods similar to those found in ReadOnlyKeyValueStore to the
> > > > > ReadOnlyWindowStore class. As it appears the discussion has
> reached a
> > > > > conclusion, I would like to start the voting process.
> > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > > 205%3A+Add+all%28%29+and+range%28%29+API+to+ReadOnlyWindowStore
> > > > >
> > > > > Thanks for your patience!
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > -- Guozhang
> > > >
> > >
> >
> >
> >
> > --
> > -- Guozhang
> >
>
Re: [VOTE] KIP-205: Add all() and range() API to ReadOnlyWindowStore
Posted by Bill Bejeck <bb...@gmail.com>.
+1 for the KIP and +1 on renaming "range" to "fetchAll"
Thanks,
Bill
On Thu, Oct 26, 2017 at 4:39 PM, Guozhang Wang <wa...@gmail.com> wrote:
> BTW, Xavier has an argument to rename "range" as it does not take a key or
> key range at all. I'd propose we rename this function to "fetchAll".
>
> Guozhang
>
> On Wed, Oct 25, 2017 at 7:27 AM, Damian Guy <da...@gmail.com> wrote:
>
> > +1
> >
> > On Tue, 24 Oct 2017 at 16:46 Guozhang Wang <wa...@gmail.com> wrote:
> >
> > > +1. Thanks.
> > >
> > > On Mon, Oct 23, 2017 at 8:11 PM, Richard Yu <
> yohan.richard.yu@gmail.com>
> > > wrote:
> > >
> > > > Hi all,
> > > >
> > > > I want to propose KIP-205 for the addition of new API. It is about
> > adding
> > > > methods similar to those found in ReadOnlyKeyValueStore to the
> > > > ReadOnlyWindowStore class. As it appears the discussion has reached a
> > > > conclusion, I would like to start the voting process.
> > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > > 205%3A+Add+all%28%29+and+range%28%29+API+to+ReadOnlyWindowStore
> > > >
> > > > Thanks for your patience!
> > > >
> > >
> > >
> > >
> > > --
> > > -- Guozhang
> > >
> >
>
>
>
> --
> -- Guozhang
>
Re: [VOTE] KIP-205: Add all() and range() API to ReadOnlyWindowStore
Posted by Guozhang Wang <wa...@gmail.com>.
BTW, Xavier has an argument to rename "range" as it does not take a key or
key range at all. I'd propose we rename this function to "fetchAll".
Guozhang
On Wed, Oct 25, 2017 at 7:27 AM, Damian Guy <da...@gmail.com> wrote:
> +1
>
> On Tue, 24 Oct 2017 at 16:46 Guozhang Wang <wa...@gmail.com> wrote:
>
> > +1. Thanks.
> >
> > On Mon, Oct 23, 2017 at 8:11 PM, Richard Yu <yo...@gmail.com>
> > wrote:
> >
> > > Hi all,
> > >
> > > I want to propose KIP-205 for the addition of new API. It is about
> adding
> > > methods similar to those found in ReadOnlyKeyValueStore to the
> > > ReadOnlyWindowStore class. As it appears the discussion has reached a
> > > conclusion, I would like to start the voting process.
> > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > > 205%3A+Add+all%28%29+and+range%28%29+API+to+ReadOnlyWindowStore
> > >
> > > Thanks for your patience!
> > >
> >
> >
> >
> > --
> > -- Guozhang
> >
>
--
-- Guozhang
Re: [VOTE] KIP-205: Add all() and range() API to ReadOnlyWindowStore
Posted by Damian Guy <da...@gmail.com>.
+1
On Tue, 24 Oct 2017 at 16:46 Guozhang Wang <wa...@gmail.com> wrote:
> +1. Thanks.
>
> On Mon, Oct 23, 2017 at 8:11 PM, Richard Yu <yo...@gmail.com>
> wrote:
>
> > Hi all,
> >
> > I want to propose KIP-205 for the addition of new API. It is about adding
> > methods similar to those found in ReadOnlyKeyValueStore to the
> > ReadOnlyWindowStore class. As it appears the discussion has reached a
> > conclusion, I would like to start the voting process.
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > 205%3A+Add+all%28%29+and+range%28%29+API+to+ReadOnlyWindowStore
> >
> > Thanks for your patience!
> >
>
>
>
> --
> -- Guozhang
>
Re: [VOTE] KIP-205: Add all() and range() API to ReadOnlyWindowStore
Posted by Guozhang Wang <wa...@gmail.com>.
+1. Thanks.
On Mon, Oct 23, 2017 at 8:11 PM, Richard Yu <yo...@gmail.com>
wrote:
> Hi all,
>
> I want to propose KIP-205 for the addition of new API. It is about adding
> methods similar to those found in ReadOnlyKeyValueStore to the
> ReadOnlyWindowStore class. As it appears the discussion has reached a
> conclusion, I would like to start the voting process.
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> 205%3A+Add+all%28%29+and+range%28%29+API+to+ReadOnlyWindowStore
>
> Thanks for your patience!
>
--
-- Guozhang