Skip to content

Remove thread.name from metrics #14061

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.

Sign up for GitHub

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

Merged
merged 14 commits into from
Jun 30, 2025
Merged

Remove thread.name from metrics #14061

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
1106d44
feat: remove thread.name from attributes
jhayes2-chwy Jun 18, 2025
a8508fe
feat: improve ThreadGrouper grouping logic
jhayes2-chwy Jun 18, 2025
6f666c2
fix: switch to a bounded LRU cache for the Consumer<RecordedEvent> in…
jhayes2-chwy Jun 18, 2025
b762c33
feat: remove unnecessary synchronization
jhayes2-chwy Jun 18, 2025
f9b198d
refactor: remove the cache entirely, and inline the plumbing built fo…
jhayes2-chwy Jun 18, 2025
960e102
refactor: remove ThreadGrouper as well, since it is now unused
jhayes2-chwy Jun 19, 2025
28bd154
refactor: remove redundant if-check
jhayes2-chwy Jun 20, 2025
ce515b0
./gradlew spotlessApply
otelbot[bot] Jun 20, 2025
409a69e
Merge branch 'main' into remove-thread-name-from-metrics
jhayes2-chwy Jun 25, 2025
adc293b
Merge branch 'main' into remove-thread-name-from-metrics
jhayes2-chwy Jun 26, 2025
504ce8e
Merge branch 'main' into remove-thread-name-from-metrics
jhayes2-chwy Jun 27, 2025
8835eb4
Merge branch 'main' into remove-thread-name-from-metrics
laurit Jun 27, 2025
4d0e52a
Merge branch 'main' into remove-thread-name-from-metrics
jhayes2-chwy Jun 30, 2025
29ee2b4
refactor: remove explicit no-args super ctor calls
jhayes2-chwy Jun 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 5 additions & 7 deletions ...src/main/java/io/opentelemetry/instrumentation/runtimemetrics/java17/HandlerRegistry.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
import io.opentelemetry.api.metrics.MeterBuilder;
import io.opentelemetry.instrumentation.api.internal.EmbeddedInstrumentationProperties;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.RecordedEventHandler;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.ThreadGrouper;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.buffer.DirectBufferStatisticsHandler;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.classes.ClassesLoadedHandler;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.container.ContainerConfigurationHandler;
Expand Down Expand Up @@ -84,17 +83,16 @@ static List<RecordedEventHandler> getHandlers(
}
}

ThreadGrouper grouper = new ThreadGrouper();
List<RecordedEventHandler> basicHandlers =
List.of(
new ObjectAllocationInNewTlabHandler(meter, grouper),
new ObjectAllocationOutsideTlabHandler(meter, grouper),
new NetworkReadHandler(meter, grouper),
new NetworkWriteHandler(meter, grouper),
new ObjectAllocationInNewTlabHandler(meter),
new ObjectAllocationOutsideTlabHandler(meter),
new NetworkReadHandler(meter),
new NetworkWriteHandler(meter),
new ContextSwitchRateHandler(meter),
new OverallCpuLoadHandler(meter),
new ContainerConfigurationHandler(meter),
new LongLockHandler(meter, grouper),
new LongLockHandler(meter),
new ThreadCountHandler(meter),
new ClassesLoadedHandler(meter),
new MetaspaceSummaryHandler(meter),
Expand Down
38 changes: 0 additions & 38 deletions ...etry/instrumentation/runtimemetrics/java17/internal/AbstractThreadDispatchingHandler.java
View file
Open in desktop

This file was deleted.

27 changes: 0 additions & 27 deletions ...n/java/io/opentelemetry/instrumentation/runtimemetrics/java17/internal/ThreadGrouper.java
View file
Open in desktop

This file was deleted.

46 changes: 16 additions & 30 deletions .../io/opentelemetry/instrumentation/runtimemetrics/java17/internal/cpu/LongLockHandler.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,34 +9,37 @@
import io.opentelemetry.api.metrics.DoubleHistogram;
import io.opentelemetry.api.metrics.Meter;
import io.opentelemetry.instrumentation.runtimemetrics.java17.JfrFeature;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.AbstractThreadDispatchingHandler;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.Constants;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.DurationUtil;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.ThreadGrouper;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.RecordedEventHandler;
import java.time.Duration;
import java.util.Optional;
import java.util.function.Consumer;
import jdk.jfr.consumer.RecordedEvent;

/**
* This class is internal and is hence not for public use. Its APIs are unstable and can change at
* any time.
*/
public final class LongLockHandler extends AbstractThreadDispatchingHandler {
public final class LongLockHandler implements RecordedEventHandler {
private static final String METRIC_NAME = "jvm.cpu.longlock";
private static final String METRIC_DESCRIPTION = "Long lock times";
private static final String EVENT_NAME = "jdk.JavaMonitorWait";

private static final String EVENT_THREAD = "eventThread";

private final DoubleHistogram histogram;
private final Attributes attributes;

public LongLockHandler(Meter meter, ThreadGrouper grouper) {
super(grouper);
public LongLockHandler(Meter meter) {
super();
histogram =
meter
.histogramBuilder(METRIC_NAME)
.setDescription(METRIC_DESCRIPTION)
.setUnit(Constants.SECONDS)
.build();

attributes = Attributes.empty();
}

@Override
Expand All @@ -50,34 +53,17 @@ public JfrFeature getFeature() {
}

@Override
public Consumer<RecordedEvent> createPerThreadSummarizer(String threadName) {
return new PerThreadLongLockHandler(histogram, threadName);
public void accept(RecordedEvent recordedEvent) {
if (recordedEvent.hasField(EVENT_THREAD)) {
Copy link
Contributor

@laurit laurit Jun 20, 2025

Choose a reason for hiding this comment

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

Is this check needed? I think that previously this check was always true because ThreadGrouper would have returned null if the thread was missing. I'd remove this check unless anybody sees a reason why to keep it.

Copy link
Contributor Author

@jhayes2-chwy jhayes2-chwy Jun 20, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

It looks like it may have been needed in order to add an attribute for the monitor's className, but I'd suspect that would have the same high-cardinality concerns as threadName, presuming it was used on Metrics.

Though I can personally see where that could provide a lot of value without cardinality issues, e.g. as an attribute on a Span Event or Profile. But unless there's active plans to build out that logic super soon, IMHO it'd be a premature optimization to keep the check in, at least for this specific reason.

Copy link
Contributor Author

@jhayes2-chwy jhayes2-chwy Jun 23, 2025

Choose a reason for hiding this comment

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

I have gone ahead and removed it.

histogram.record(DurationUtil.toSeconds(recordedEvent.getDuration()), attributes);
}
// What about the class name in MONITOR_CLASS ?
// We can get a stack trace from the thread on the event
// var eventThread = recordedEvent.getThread(EVENT_THREAD);
}

@Override
public Optional<Duration> getThreshold() {
return Optional.empty();
}

private static class PerThreadLongLockHandler implements Consumer<RecordedEvent> {
private static final String EVENT_THREAD = "eventThread";

private final DoubleHistogram histogram;
private final Attributes attributes;

public PerThreadLongLockHandler(DoubleHistogram histogram, String threadName) {
this.histogram = histogram;
this.attributes = Attributes.of(Constants.ATTR_THREAD_NAME, threadName);
}

@Override
public void accept(RecordedEvent recordedEvent) {
if (recordedEvent.hasField(EVENT_THREAD)) {
histogram.record(DurationUtil.toSeconds(recordedEvent.getDuration()), attributes);
}
// What about the class name in MONITOR_CLASS ?
// We can get a stack trace from the thread on the event
// var eventThread = recordedEvent.getThread(EVENT_THREAD);
}
}
}
41 changes: 11 additions & 30 deletions ...strumentation/runtimemetrics/java17/internal/memory/ObjectAllocationInNewTlabHandler.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,8 @@
import io.opentelemetry.api.metrics.LongHistogram;
import io.opentelemetry.api.metrics.Meter;
import io.opentelemetry.instrumentation.runtimemetrics.java17.JfrFeature;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.AbstractThreadDispatchingHandler;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.Constants;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.ThreadGrouper;
import java.util.function.Consumer;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.RecordedEventHandler;
import jdk.jfr.consumer.RecordedEvent;

/**
Expand All @@ -22,20 +20,23 @@
* <p>This class is internal and is hence not for public use. Its APIs are unstable and can change
* at any time.
*/
public final class ObjectAllocationInNewTlabHandler extends AbstractThreadDispatchingHandler {
public final class ObjectAllocationInNewTlabHandler implements RecordedEventHandler {
private static final String EVENT_NAME = "jdk.ObjectAllocationInNewTLAB";
private static final String TLAB_SIZE = "tlabSize";

private final LongHistogram histogram;
private final Attributes attributes;

public ObjectAllocationInNewTlabHandler(Meter meter, ThreadGrouper grouper) {
super(grouper);
public ObjectAllocationInNewTlabHandler(Meter meter) {
super();
histogram =
meter
.histogramBuilder(Constants.METRIC_NAME_MEMORY_ALLOCATION)
.setDescription(Constants.METRIC_DESCRIPTION_MEMORY_ALLOCATION)
.setUnit(Constants.BYTES)
.ofLongs()
.build();
attributes = Attributes.of(Constants.ATTR_ARENA_NAME, "TLAB");
}

@Override
Expand All @@ -49,29 +50,9 @@ public JfrFeature getFeature() {
}

@Override
public Consumer<RecordedEvent> createPerThreadSummarizer(String threadName) {
return new PerThreadObjectAllocationInNewTlabHandler(histogram, threadName);
}

/** This class aggregates all TLAB allocation JFR events for a single thread */
private static class PerThreadObjectAllocationInNewTlabHandler
implements Consumer<RecordedEvent> {
private static final String TLAB_SIZE = "tlabSize";

private final LongHistogram histogram;
private final Attributes attributes;

public PerThreadObjectAllocationInNewTlabHandler(LongHistogram histogram, String threadName) {
this.histogram = histogram;
this.attributes =
Attributes.of(Constants.ATTR_THREAD_NAME, threadName, Constants.ATTR_ARENA_NAME, "TLAB");
}

@Override
public void accept(RecordedEvent ev) {
histogram.record(ev.getLong(TLAB_SIZE), attributes);
// Probably too high a cardinality
// ev.getClass("objectClass").getName();
}
public void accept(RecordedEvent ev) {
histogram.record(ev.getLong(TLAB_SIZE), attributes);
// Probably too high a cardinality
// ev.getClass("objectClass").getName();
}
}
42 changes: 12 additions & 30 deletions ...rumentation/runtimemetrics/java17/internal/memory/ObjectAllocationOutsideTlabHandler.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,8 @@
import io.opentelemetry.api.metrics.LongHistogram;
import io.opentelemetry.api.metrics.Meter;
import io.opentelemetry.instrumentation.runtimemetrics.java17.JfrFeature;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.AbstractThreadDispatchingHandler;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.Constants;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.ThreadGrouper;
import java.util.function.Consumer;
import io.opentelemetry.instrumentation.runtimemetrics.java17.internal.RecordedEventHandler;
import jdk.jfr.consumer.RecordedEvent;

/**
Expand All @@ -22,20 +20,24 @@
* <p>This class is internal and is hence not for public use. Its APIs are unstable and can change
* at any time.
*/
public final class ObjectAllocationOutsideTlabHandler extends AbstractThreadDispatchingHandler {
public final class ObjectAllocationOutsideTlabHandler implements RecordedEventHandler {
private static final String EVENT_NAME = "jdk.ObjectAllocationOutsideTLAB";
private static final String ALLOCATION_SIZE = "allocationSize";

private final LongHistogram histogram;
private final Attributes attributes;

public ObjectAllocationOutsideTlabHandler(Meter meter, ThreadGrouper grouper) {
super(grouper);
public ObjectAllocationOutsideTlabHandler(Meter meter) {
super();
histogram =
meter
.histogramBuilder(Constants.METRIC_NAME_MEMORY_ALLOCATION)
.setDescription(Constants.METRIC_DESCRIPTION_MEMORY_ALLOCATION)
.setUnit(Constants.BYTES)
.ofLongs()
.build();

attributes = Attributes.of(Constants.ATTR_ARENA_NAME, "Main");
}

@Override
Expand All @@ -49,29 +51,9 @@ public JfrFeature getFeature() {
}

@Override
public Consumer<RecordedEvent> createPerThreadSummarizer(String threadName) {
return new PerThreadObjectAllocationOutsideTlabHandler(histogram, threadName);
}

/** This class aggregates all non-TLAB allocation JFR events for a single thread */
private static class PerThreadObjectAllocationOutsideTlabHandler
implements Consumer<RecordedEvent> {
private static final String ALLOCATION_SIZE = "allocationSize";

private final LongHistogram histogram;
private final Attributes attributes;

public PerThreadObjectAllocationOutsideTlabHandler(LongHistogram histogram, String threadName) {
this.histogram = histogram;
this.attributes =
Attributes.of(Constants.ATTR_THREAD_NAME, threadName, Constants.ATTR_ARENA_NAME, "Main");
}

@Override
public void accept(RecordedEvent ev) {
histogram.record(ev.getLong(ALLOCATION_SIZE), attributes);
// Probably too high a cardinality
// ev.getClass("objectClass").getName();
}
public void accept(RecordedEvent ev) {
histogram.record(ev.getLong(ALLOCATION_SIZE), attributes);
// Probably too high a cardinality
// ev.getClass("objectClass").getName();
}
}
Loading
Loading