Hello Matthias, I added here since in the original KIP-345 this API was brought up once for the similar purpose. I agree this may cause more confusions than clarifications so I'll remove this.
Guozhang On Sat, Jan 12, 2019 at 6:30 PM Matthias J. Sax <matth...@confluent.io> wrote: > Thanks for the KIP. > > One side comment about rejected alternatives. I would remove the > sentence, because `StreamsMetadata` is part of IQ feature and not > related at all to this change -- thus, it does not seem to be a valid > alternative. > > > +1 (binding) > > > -Matthias > > On 1/11/19 5:47 AM, Bill Bejeck wrote: > > Thanks for the KIP Guozhang, > > > > +1 for me. > > > > On Fri, Jan 11, 2019 at 7:10 AM Damian Guy <damian....@gmail.com> wrote: > > > >> +1 > >> > >> On Fri, 11 Jan 2019 at 05:09, John Roesler <j...@confluent.io> wrote: > >> > >>> Hi Guozhang, > >>> > >>> It sounds reasonable to me. I'm +1 (nonbinding). > >>> > >>> -John > >>> > >>> On Tue, Jan 8, 2019 at 8:51 PM Guozhang Wang <wangg...@gmail.com> > wrote: > >>> > >>>> Hello folks, > >>>> > >>>> I'd like to start a voting process for the following KIP: > >>>> > >>>> > >>>> > >>> > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-414%3A+Expose+Embedded+ClientIds+in+Kafka+Streams > >>>> > >>>> It is a pretty straight-forward and small augment to Stream's public > >>>> ThreadMetadata interface, as an outcome of the discussion on KIP-345. > >>> Hence > >>>> I think we can skip the DISCUSS thread and move on to voting directly. > >> If > >>>> people have any questions about the context of KIP-414 or KIP-345, > >> please > >>>> feel free to read these two wiki pages and let me know. > >>>> > >>>> > >>>> -- Guozhang > >>>> > >>> > >> > > > > -- -- Guozhang
