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 10, 2025 at 10:45 AM Kevin Wu <kevin.wu2...@gmail.com> wrote: > 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 > > > > > >
