+1 On 9/1/17 2:53 PM, Bill Bejeck wrote: > +1 > > On Thu, Aug 31, 2017 at 10:20 AM, Damian Guy <damian....@gmail.com> wrote: > >> Thanks everyone for voting! Unfortunately i've had to make a bit of an >> update based on some issues found during implementation. >> The main changes are: >> BytesStoreSupplier -> StoreSupplier >> Addition of: >> WindowBytesStoreSupplier, KeyValueBytesStoreSupplier, >> SessionBytesStoreSupplier that will restrict store types to <Bytes, byte[]> >> 3 new overloads added to Materialized to enable developers to create a >> Materialized of the appropriate type, i..e, WindowStore etc >> Update DSL where Materialized is used such that the stores have generic >> types of <Bytes, byte[]> >> Some minor changes to the arguments to Store#persistentWindowStore and >> Store#persistentSessionStore >> >> Please take a look and recast the votes. >> >> Thanks for your time, >> Damian >> >> On Fri, 25 Aug 2017 at 17:05 Matthias J. Sax <matth...@confluent.io> >> wrote: >> >>> Thanks Damian. Great KIP! >>> >>> +1 >>> >>> >>> -Matthias >>> >>> On 8/25/17 6:45 AM, Damian Guy wrote: >>>> Hi, >>>> >>>> I've just realised we need to add two methods to StateStoreBuilder or >> it >>>> isn't going to work: >>>> >>>> Map<String, String> logConfig(); >>>> boolean loggingEnabled(); >>>> >>>> These are needed when we are building the topology and determining >>>> changelog topic names and configs. >>>> >>>> >>>> I've also update the KIP to add >>>> >>>> StreamBuilder#stream(String topic) >>>> >>>> StreamBuilder#stream(String topic, Consumed options) >>>> >>>> >>>> Thanks >>>> >>>> >>>> On Thu, 24 Aug 2017 at 22:11 Sriram Subramanian <r...@confluent.io> >>> wrote: >>>> >>>>> +1 >>>>> >>>>> On Thu, Aug 24, 2017 at 10:20 AM, Guozhang Wang <wangg...@gmail.com> >>>>> wrote: >>>>> >>>>>> +1. Thanks Damian! >>>>>> >>>>>> On Thu, Aug 24, 2017 at 9:47 AM, Bill Bejeck <bbej...@gmail.com> >>> wrote: >>>>>> >>>>>>> Thanks for the KIP! >>>>>>> >>>>>>> +1 >>>>>>> >>>>>>> Thanks, >>>>>>> Bill >>>>>>> >>>>>>> On Thu, Aug 24, 2017 at 12:25 PM, Damian Guy <damian....@gmail.com> >>>>>> wrote: >>>>>>> >>>>>>>> Hi, >>>>>>>> >>>>>>>> I'd like to kick off the voting thread for KIP-182: >>>>>>>> https://cwiki.apache.org/confluence/display/KAFKA/KIP- >>>>>>>> 182%3A+Reduce+Streams+DSL+overloads+and+allow+easier+ >>>>>>>> use+of+custom+storage+engines >>>>>>>> >>>>>>>> Thanks, >>>>>>>> Damian >>>>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> -- Guozhang >>>>>> >>>>> >>>> >>> >>> >> >
signature.asc
Description: OpenPGP digital signature