@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 >> >
