Thanks Guozhang. Thanks! Sagar.
On Mon, Feb 14, 2022 at 7:40 AM Guozhang Wang <wangg...@gmail.com> wrote: > Hi Sagar, > > Looks good to define it in the NamedCacheMetrics. Though since this is an > internal implementation detail and neither of the classes are public, we do > not actually need to define it in the KIP :) > > > Guozhang > > On Sat, Feb 12, 2022 at 4:22 AM Sagar <sagarmeansoc...@gmail.com> wrote: > > > @Guozhang, > > > > I have sent an update to this KIP. I have a question though.. Should this > > new metric be defined in TaskMetrics level or NamedCacheMetrics? I think > > the latter makes sense as that holds the cache size at a task level and > > exposes some other cache related metrics as well like hit-ratio. > > > > Thanks! > > Sagar. > > > > > > On Sat, Feb 12, 2022 at 1:14 PM Sagar <sagarmeansoc...@gmail.com> wrote: > > > > > Hi All, > > > > > > There's another amendment proposed for this KIP. We are adding a new > > > metric type called *cache-size-bytes-total *to capture the cache size > in > > > bytes accumulated by a task. > > > > > > > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=186878390 > > > > > > Thanks! > > > Sagar. > > > > > > On Mon, Jan 24, 2022 at 7:55 AM Guozhang Wang <wangg...@gmail.com> > > wrote: > > > > > >> Thanks Sagar, I'm +1 on the renamed metric. > > >> > > >> On Sat, Jan 22, 2022 at 6:56 PM Sagar <sagarmeansoc...@gmail.com> > > wrote: > > >> > > >> > Hi All, > > >> > > > >> > There is a small update to the KIP whereby the newly introduced > metric > > >> > *total-bytes > > >> > *has been renamed to *input-buffer-bytes-total.* > > >> > > > >> > Thanks! > > >> > Sagar. > > >> > > > >> > On Wed, Sep 29, 2021 at 9:57 AM Sagar <sagarmeansoc...@gmail.com> > > >> wrote: > > >> > > > >> > > We have 3 binding votes: Sophie/Guozhang/Mathias > > >> > > and 2 non-binding votes: Josep/Luke and no -1 votes. > > >> > > > > >> > > Marking this KIP as accepted. > > >> > > > > >> > > Thanks everyone! > > >> > > > > >> > > Thanks! > > >> > > Sagar. > > >> > > > > >> > > > > >> > > > > >> > > On Wed, Sep 29, 2021 at 7:18 AM Matthias J. Sax <mj...@apache.org > > > > >> > wrote: > > >> > > > > >> > >> +1 (binding) > > >> > >> > > >> > >> On 9/28/21 10:40 AM, Sagar wrote: > > >> > >> > Hi All, > > >> > >> > > > >> > >> > Bumping this vote thread again! > > >> > >> > > > >> > >> > Thanks! > > >> > >> > Sagar. > > >> > >> > > > >> > >> > On Wed, Sep 8, 2021 at 1:19 PM Luke Chen <show...@gmail.com> > > >> wrote: > > >> > >> > > > >> > >> >> Thanks for the KIP. > > >> > >> >> > > >> > >> >> + 1 (non-binding) > > >> > >> >> > > >> > >> >> Thanks. > > >> > >> >> Luke > > >> > >> >> > > >> > >> >> On Wed, Sep 8, 2021 at 2:48 PM Josep Prat > > >> > <josep.p...@aiven.io.invalid > > >> > >> > > > >> > >> >> wrote: > > >> > >> >> > > >> > >> >>> +1 (non binding). > > >> > >> >>> > > >> > >> >>> Thanks for the KIP Sagar! > > >> > >> >>> ——— > > >> > >> >>> Josep Prat > > >> > >> >>> > > >> > >> >>> Aiven Deutschland GmbH > > >> > >> >>> > > >> > >> >>> Immanuelkirchstraße 26, 10405 Berlin > > >> > >> >>> > > >> > >> >>> Amtsgericht Charlottenburg, HRB 209739 B > > >> > >> >>> > > >> > >> >>> Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen > > >> > >> >>> > > >> > >> >>> m: +491715557497 > > >> > >> >>> > > >> > >> >>> w: aiven.io > > >> > >> >>> > > >> > >> >>> e: josep.p...@aiven.io > > >> > >> >>> > > >> > >> >>> On Wed, Sep 8, 2021, 03:29 Sophie Blee-Goldman > > >> > >> >> <sop...@confluent.io.invalid > > >> > >> >>>> > > >> > >> >>> wrote: > > >> > >> >>> > > >> > >> >>>> +1 (binding) > > >> > >> >>>> > > >> > >> >>>> Thanks for the KIP! > > >> > >> >>>> > > >> > >> >>>> -Sophie > > >> > >> >>>> > > >> > >> >>>> On Tue, Sep 7, 2021 at 1:59 PM Guozhang Wang < > > >> wangg...@gmail.com> > > >> > >> >> wrote: > > >> > >> >>>> > > >> > >> >>>>> Thanks Sagar, +1 from me. > > >> > >> >>>>> > > >> > >> >>>>> > > >> > >> >>>>> Guozhang > > >> > >> >>>>> > > >> > >> >>>>> On Sat, Sep 4, 2021 at 10:29 AM Sagar < > > >> sagarmeansoc...@gmail.com> > > >> > >> >>> wrote: > > >> > >> >>>>> > > >> > >> >>>>>> Hi All, > > >> > >> >>>>>> > > >> > >> >>>>>> I would like to start a vote on the following KIP: > > >> > >> >>>>>> > > >> > >> >>>>>> > > >> > >> >>>>> > > >> > >> >>>> > > >> > >> >>> > > >> > >> >> > > >> > >> > > >> > > > >> > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=186878390 > > >> > >> >>>>>> > > >> > >> >>>>>> Thanks! > > >> > >> >>>>>> Sagar. > > >> > >> >>>>>> > > >> > >> >>>>> > > >> > >> >>>>> > > >> > >> >>>>> -- > > >> > >> >>>>> -- Guozhang > > >> > >> >>>>> > > >> > >> >>>> > > >> > >> >>> > > >> > >> >> > > >> > >> > > > >> > >> > > >> > > > > >> > > > >> > > >> > > >> -- > > >> -- Guozhang > > >> > > > > > > > > -- > -- Guozhang >
