Hi Mahsa, Thanks for the KIP. I think there was an issue with my original reply since it is not showing up on the thread. Trying again.
In the Motivation section, can we state why the current metrics involving the controller's event queue thread -- time spent in the queue and process time -- are not sufficient? Can we also match the naming style of those other event queue metrics for consistency (i.e. the type should be ControllerEventManager)? I think it would also be helpful to explain how this proposed metric will be monitored by the operator. Best, Kevin Wu On 2025/07/07 20:05:22 Jonah Hooper wrote: > Thanks for the KIP, Mahsa. > > Have one initial question: > > > The ratio of time the controller thread is idle relative to the total time > > (idle+active). > > > How is the active and idle time calculated? Is it in total over the time > period in which the controller is active? Or is there a specific window > period? > > Best, > Jonah Hooper > > > On Thu, Jul 3, 2025 at 4:41 PM Mahsa Seifikar > <ms...@confluent.io.invalid> 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 > > >