Thank you, Roger. I have changed the naming of the internal classes slightly and updated the example. The new CompoundStat is called Meter and it combines Rate and Count (the aggregate), leaving Rate as a simple stat. Will submit an initial PR to help with the review.
On Thu, Aug 17, 2017 at 12:09 AM, Roger Hoover <roger.hoo...@gmail.com> wrote: > Rajini, > > Thank you. This is very useful. Grouping by metric by prefixing the name > instead of making them MBeans is not quite as nice but seems like an good > compromise for backward compatibility. > > Cheers, > > Roger > > On Wed, Aug 16, 2017 at 5:35 AM, Rajini Sivaram <rajinisiva...@gmail.com> > wrote: > > > Sorry, pressed send by mistake. > > > > The KIP is here: > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > 187+-+Add+cumulative+count+metric+for+all+Kafka+rate+metrics > > > > Feedback and suggestions are appreciated. > > > > Thank you... > > > > Rajini > > > > > > On Wed, Aug 16, 2017 at 1:34 PM, Rajini Sivaram <rajinisiva...@gmail.com > > > > wrote: > > > > > Hi all, > > > > > > I have posted a KIP to add a cumulative count attribute for all Kafka > > rate > > > metrics to make downstream processing simpler, more accurate and > > flexible: > > > > > > > > >
