Thanks Jonah and Kevin for the feedback. I have updated the KIP accordingly. We ideally want to use something like the TimeRatio type for this metric, similar to how "poll-idle-ratio" is measured in KafkaRaftMetrics.
Please let me know if you have any further feedback. Best, Mahsa Seifikar On Fri, Jul 11, 2025 at 4:08 PM Kevin Wu <kevin.wu2...@gmail.com> wrote: > Hi Mahsa and Jonah, > > Since we're adding this new metric to a metrics group that is still using > Yammer, ideally I think we want to use RatioGauge to give us the sampling > functionality we need. It's possible that we can get similar functionality > from Histogram, which I know other Yammer metrics in Kafka use. We are > still able to get gauge metrics from the histogram, as they are the most > straightforward for the operator to monitor (e.g. if the metric value > X, > alert). For example, metrics that are histograms, like EventQueueTimeMs, > are often monitored via their p99 or p999 value. > > There are several other "thread-idle-ratio" metrics in Kafka, but those are > all using the newer, internal KafkaMetrics library's sensors. > > Best, > Kevin Wu > > On Thu, Jul 10, 2025 at 2:09 PM Mahsa Seifikar > <mseifi...@confluent.io.invalid> wrote: > > > Hi Jonah and Kevin, > > > > Thanks for your comments. I have now updated the KIP to address your > > feedback. > > > > Please let me know if you have any further questions. > > > > Best, > > Mahsa Seifikar > > > > On Thu, Jul 3, 2025 at 4:40 PM Mahsa Seifikar <mseifi...@confluent.io> > > wrote: > > > > > Hello all, > > > > > > I wrote a short KIP to add a new metric for controller thread idleness. > > > > > > Here is the link: > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-1190%3A+Add+a+metric+for+controller+thread+idleness > > > > > > Thanks, > > > Mahsa Seifikar > > > > > > > > > > > >
