Thank Vicky! I'm +1. Guozhang
On Mon, Sep 12, 2022 at 7:02 PM John Roesler <vvcep...@apache.org> wrote: > Thanks for the updates, Vicky! > > I've reviewed the KIP and your POC PR, > and I'm +1 (binding). > > Thanks! > -John > > On Mon, Sep 12, 2022, at 09:13, Vasiliki Papavasileiou wrote: > > Hey Guozhang, > > > > Great suggestion, I made the change. > > > > Best, > > Vicky > > > > On Fri, Sep 9, 2022 at 10:43 PM Guozhang Wang <wangg...@gmail.com> > wrote: > > > >> Thanks Vicky, that reads much clearer now. > >> > >> Just regarding the value string name itself: "self.join" may be > confusing > >> compared to other values that people would think before this config is > >> enabled, self-join are not allowed at all. Maybe we can rename it to > >> "single.store.self.join"? > >> > >> Guozhang > >> > >> On Fri, Sep 9, 2022 at 2:15 AM Vasiliki Papavasileiou > >> <vpapavasile...@confluent.io.invalid> wrote: > >> > >> > Hey Guozhang, > >> > > >> > Ah it seems my text was not very clear :) > >> > With "TOPOLOGY_OPTIMIZATION_CONFIG will be extended to accept a list > of > >> > optimization rule configs" I meant that it will accept the new value > >> > strings for each optimization rule. Let me rephrase that in the KIP to > >> make > >> > it clearer. > >> > Is it better now? > >> > > >> > Best, > >> > Vicky > >> > > >> > On Thu, Sep 8, 2022 at 9:07 PM Guozhang Wang <wangg...@gmail.com> > wrote: > >> > > >> > > Thanks Vicky, > >> > > > >> > > I read through the KIP again and it looks good to me. Just a quick > >> > question > >> > > regarding the public config changes: you mentioned "No public > >> interfaces > >> > > will be impacted. The config TOPOLOGY_OPTIMIZATION_CONFIG will be > >> > extended > >> > > to accept a list of optimization rule configs in addition to the > global > >> > > values "all" and "none" . But there are no new value strings > mentioned > >> in > >> > > this KIP, so that means we will apply this optimization only when > `all` > >> > is > >> > > specified in the config right? > >> > > > >> > > > >> > > Guozhang > >> > > > >> > > > >> > > On Thu, Sep 8, 2022 at 12:02 PM Vasiliki Papavasileiou > >> > > <vpapavasile...@confluent.io.invalid> wrote: > >> > > > >> > > > Hello everyone, > >> > > > > >> > > > I'd like to open the vote for KIP-862, which proposes to optimize > >> > > > stream-stream self-joins by using a single state store for the > join. > >> > > > > >> > > > The proposal is here: > >> > > > > >> > > > > >> > > > >> > > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-862%3A+Self-join+optimization+for+stream-stream+joins > >> > > > > >> > > > Thanks to all who reviewed the proposal, and thanks in advance for > >> > taking > >> > > > the time to vote! > >> > > > > >> > > > Thank you, > >> > > > Vicky > >> > > > > >> > > > >> > > > >> > > -- > >> > > -- Guozhang > >> > > > >> > > >> > >> > >> -- > >> -- Guozhang > >> > -- -- Guozhang
