-
Notifications
You must be signed in to change notification settings - Fork 13.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KAFKA-15669: Implement telemetry metric naming strategy (KIP-714) #14619
Conversation
...ts/src/test/java/org/apache/kafka/common/telemetry/internals/MetricNamingConventionTest.java
Outdated
Show resolved
Hide resolved
Depends on PR - #14575 for build. |
clients/src/main/java/org/apache/kafka/common/telemetry/internals/MetricNamingConvention.java
Outdated
Show resolved
Hide resolved
clients/src/main/java/org/apache/kafka/common/telemetry/internals/MetricNamingConvention.java
Outdated
Show resolved
Hide resolved
clients/src/main/java/org/apache/kafka/common/telemetry/internals/MetricNamingConvention.java
Outdated
Show resolved
Hide resolved
clients/src/main/java/org/apache/kafka/common/telemetry/internals/MetricNamingConvention.java
Outdated
Show resolved
Hide resolved
...ts/src/test/java/org/apache/kafka/common/telemetry/internals/MetricNamingConventionTest.java
Outdated
Show resolved
Hide resolved
clients/src/main/java/org/apache/kafka/common/telemetry/internals/MetricNamingConvention.java
Outdated
Show resolved
Hide resolved
clients/src/main/java/org/apache/kafka/common/telemetry/internals/MetricNamingConvention.java
Outdated
Show resolved
Hide resolved
clients/src/main/java/org/apache/kafka/common/telemetry/internals/MetricNamingConvention.java
Outdated
Show resolved
Hide resolved
* This class encapsulates naming and mapping conventions defined as part of | ||
* <a href="https://cwiki.apache.org/confluence/display/KAFKA/KIP-714%3A+Client+metrics+and+observability#KIP714:Clientmetricsandobservability-Metricsnamingandformat">Metrics naming and format</a> | ||
*/ | ||
public class MetricNamingConvention { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe a more explicit class name would help here. Since the goal of this mapping is to map the metric names to something that fits OpenTelemetry conventions, maybe we can call this OpenTelemetryMetricNamingConvention ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed name to TelemtryMetricNamingConvention
as we generally referred by same in KIP. Let me know if it works or we still feel that the name should be more specific as you suggested.
...ts/src/test/java/org/apache/kafka/common/telemetry/internals/MetricNamingConventionTest.java
Outdated
Show resolved
Hide resolved
The tests failing are not related to this PR |
…ache#14619) The PR defines the naming convention for telemetry metric names for KIP-714 - jira. Telemetry metric name should be dot separated and tags should be snake case. PR adds the interface which will be used in MetricsReporter implementation to construct metric names. Reviewers: Xavier Léauté <[email protected]>, Walker Carlson <[email protected]>, Matthias J. Sax <[email protected]>, Andrew Schofield <[email protected]>
…ache#14619) The PR defines the naming convention for telemetry metric names for KIP-714 - jira. Telemetry metric name should be dot separated and tags should be snake case. PR adds the interface which will be used in MetricsReporter implementation to construct metric names. Reviewers: Xavier Léauté <[email protected]>, Walker Carlson <[email protected]>, Matthias J. Sax <[email protected]>, Andrew Schofield <[email protected]>
The PR defines the naming convention for telemetry metric names for KIP-714 - jira. Telemetry metric name should be dot separated and tags should be snake case.
PR adds the interface which will be used in MetricsReporter implementation to construct metric names.
Committer Checklist (excluded from commit message)