nicktelford commented on PR #16041:
URL: https://github.com/apache/kafka/pull/16041#issuecomment-2126919697

   @mjsax This is the final PR to add the KIP-989 metrics. Although I'm 
considering opening two, much smaller PRs:
   
   1. To add some additional tests to ensure we cover all the Iterator 
constructors
   2. To eliminate some redundancy now all the metrics have landed. Notably, we 
can replace the `AtomicInteger` used by `num-open-iterators` with just counting 
the cardinality of the `Set<MeteredIterator>` that we maintain for 
`oldest-iterator-open-since-ms`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to