Thanks for all your votes.

During the implementation, I found it'd be better to have helper methods in
`Stores`, to be able to get the store supplier by the store type:



*public static SessionBytesStoreSupplier
sessionStoreSupplierByStoreType()public static WindowBytesStoreSupplier
windowStoreSupplierByStoreType()public static KeyValueBytesStoreSupplier
keyValueStoreSupplierByStoreType()*

I've also updated in the KIP.
Please let me know if you other thoughts.

Also, welcome to vote for this KIP.

Thank you.
Luke


On Fri, Jan 21, 2022 at 4:39 AM Walker Carlson
<wcarl...@confluent.io.invalid> wrote:

> +1 non binding
>
> On Thu, Jan 20, 2022 at 2:00 PM Matthias J. Sax <mj...@apache.org> wrote:
>
> > +1 (binding)
> >
> > On 1/20/22 10:52 AM, Guozhang Wang wrote:
> > > Thanks Luke! I'm +1 on the KIP.
> > >
> > >
> > > Guozhang
> > >
> > > On Wed, Jan 19, 2022 at 5:58 PM Luke Chen <show...@gmail.com> wrote:
> > >
> > >> Hi devs,
> > >>
> > >> I'd like to start a vote for the KIP-591: Add Kafka Streams config to
> > set
> > >> default state store. The goal is to allow users to set a default store
> > in
> > >> the config, so it can apply to all the streams.
> > >>
> > >> Detailed description can be found here:
> > >>
> > >>
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-591%3A+Add+Kafka+Streams+config+to+set+default+state+store
> > >>
> > >>
> > >> Thank you.
> > >> Luke
> > >>
> > >
> > >
> >
>

Reply via email to