Skip to content

Commit 29f8b09

Browse files
authored
Clear context class loader from started threads (#7488)
1 parent a00f6a8 commit 29f8b09

File tree

3 files changed

+3
-0
lines changed

3 files changed

+3
-0
lines changed

context/src/main/java/io/opentelemetry/context/StrictContextStorage.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -252,6 +252,7 @@ static PendingScopes create() {
252252
thread.setName("weak-ref-cleaner-strictcontextstorage");
253253
thread.setPriority(Thread.MIN_PRIORITY);
254254
thread.setDaemon(true);
255+
thread.setContextClassLoader(null);
255256
thread.start();
256257
}
257258

sdk-extensions/incubator/src/main/java/io/opentelemetry/sdk/extension/incubator/trace/LeakDetectingSpanProcessor.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,7 @@ private static PendingSpans create(BiConsumer<String, Throwable> reporter) {
8787
thread.setName("weak-ref-cleaner-leakingspandetector");
8888
thread.setPriority(Thread.MIN_PRIORITY);
8989
thread.setDaemon(true);
90+
thread.setContextClassLoader(null);
9091
thread.start();
9192
return pendingSpans;
9293
}

sdk/common/src/main/java/io/opentelemetry/sdk/internal/DaemonThreadFactory.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ public Thread newThread(Runnable runnable) {
4545
try {
4646
t.setDaemon(true);
4747
t.setName(namePrefix + "-" + counter.incrementAndGet());
48+
t.setContextClassLoader(null);
4849
} catch (SecurityException e) {
4950
// Well, we tried.
5051
}

0 commit comments

Comments
 (0)