Thanks for the comments, Matthias!

I've updated the table to contain the new JMX metric and the correct MBean
name.

Regards,
Bill

On Fri, Nov 1, 2024 at 10:27 PM Matthias J. Sax <mj...@apache.org> wrote:

> Thanks for the KIP Bill. Very useful addition to be able to collect all
> metrics broker side.
>
> The KIP proposes to add a new thread-level metric for the thread state
> similar to the existing client "state" metric. The KIP lacks the
> corresponding definition though -- it only list the three new GAUGE
> metrics, but not the new thread-level STRING metric. Can you add the
> missing row to the table?
>
>
>
> > Note that the "Name" column is the MBean name.
>
> Well, not really. Of course, all information is there, but the "Name" is
> not the MBean name. The MBean is a concatenation of tags; cf
> https://kafka.apache.org/documentation/#kafka_streams_client_monitoring
> -- the MBean name is the last column, eg:
>
>      kafka.streams:type=stream-metrics,client-id=([-.\w]+)
>
>
> -Matthias
>
> On 10/31/24 3:43 PM, Bill Bejeck wrote:
> > Hi Apoorv,
> >
> > Thanks for the comments.  I've updated the table to include the
> information
> > you requested.  Note that the "Name" column is the MBean name.
> >
> > Regards,
> > Bill
> >
> >
> >
> > On Tue, Oct 29, 2024 at 1:00 PM Apoorv Mittal <apoorvmitta...@gmail.com>
> > wrote:
> >
> >> Hi Bill,
> >> Thanks for the KIP. I have a request, can we please define MBean and
> >> telemetry names for the new metrics so it's easier to see where they
> will
> >> be added (group, tags etc.)
> >>
> >> Regards,
> >> Apoorv Mittal
> >>
> >>
> >> On Tue, Oct 29, 2024 at 5:15 AM Sophie Blee-Goldman <
> sop...@responsive.dev
> >>>
> >> wrote:
> >>
> >>> Hey Bill,
> >>>
> >>> Thanks for the KIP! That all makes sense to me, just one minor note:
> >> while
> >>> you mentioned the TRACE recording level in the Motivation section, it
> >> seems
> >>> to be missing from the table in the Public Interfaces section. I assume
> >>> this will also be included, presumably with a value of 2?
> >>>
> >>> Cheers,
> >>> Sophie
> >>>
> >>> On Fri, Oct 25, 2024 at 12:20 PM Bill Bejeck <bbej...@apache.org>
> wrote:
> >>>
> >>>> Hi All,
> >>>>
> >>>> I would like to start a discussion thread on KIP-1091:Improved Kafka
> >>>> Streams operator metrics
> >>>>
> >>>> Here's a link to the KIP:
> >>>>
> >>>>
> >>>
> >>
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-1091%3A+Improved+Kafka+Streams+operator+metrics
> >>>>
> >>>> I look forward to the discussion.
> >>>>
> >>>> Thanks,
> >>>> Bill
> >>>>
> >>>
> >>
> >
>

Reply via email to