Skip to content

Conversation

@gensericghiro
Copy link
Contributor

@gensericghiro gensericghiro commented Oct 31, 2025

Summary

Updating the docs for the client state metric as a follow up to #20766

@github-actions github-actions bot added triage PRs from the community docs small Small PRs labels Oct 31, 2025
docs/ops.html Outdated
<td>thread-state</td>
<td>The state of the thread as a number (<code>ordinal()</code> of the corresponding enum).</td>
<td>kafka.streams:type=stream-thread-metrics,thread-id=([-.\w]+),process-id=([-.\w]+)</td>
<td>kafka.streams:type=stream-thread-metrics,thread-id=([-.\w]+),process-id=([-.\w]+),application-id=([-.\w]+)</td>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems we only add the new tag to client-state but not to thead-state -- so this change seems not to apply.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a follow-up to this comment, the thread-id will already have the application-id embedded in it.

@mjsax mjsax added streams kip Requires or implements a KIP and removed triage PRs from the community labels Nov 3, 2025
Copy link
Member

@bbejeck bbejeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @gensericghiro, overall lgtm once we address the outstanding comment.

Copy link
Member

@mjsax mjsax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should also add a new bullet point to the upgrade guide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved docs kip Requires or implements a KIP small Small PRs streams

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants