Just FYI, there is another effort trying to fix a similar issue from the client 
side, but is shooting for a slight different approach following KIP-187 
(https://cwiki.apache.org/confluence/display/KAFKA/KIP-187+-+Add+cumulative+count+metric+for+all+Kafka+rate+metrics).

I'm going to merge this PR (the cumulative count) as-is and will leave it to 
@vvcephei to consider for aligning the efforts in a follow-up PR potentially. 
And also while discussing with @vvcephei I realized there are a few other 
caller bugs on recording total metrics (this is not a bug for the sensor / 
metrics package itself, just the way to call `record` on the caller is 
incorrect), which I will try to fix in a separate PR as well.

[ Full content available at: https://github.com/apache/kafka/pull/5467 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to