Skip to content

Commit 898d9c3

Browse files
committed
Revert "feat(profiler): start QueueTime timer generically in AdviceUtils.capture()"
This reverts commit b3628dc.
1 parent d4c41e1 commit 898d9c3

1 file changed

Lines changed: 1 addition & 8 deletions

File tree

  • dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/instrumentation/java/concurrent

dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/instrumentation/java/concurrent/AdviceUtils.java

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,14 +58,7 @@ public static <T> void capture(ContextStore<T, State> contextStore, T task) {
5858
state = State.FACTORY.create();
5959
contextStore.put(task, state);
6060
}
61-
if (state.captureAndSetContinuation(span)) {
62-
// Start queue timing generically for all cross-thread span propagations.
63-
// Per-executor instrumentations may overwrite this with richer metadata (scheduler/queue
64-
// class, queue length) via their own startQueuingTimer calls, which is fine — the
65-
// isTimed() guard in Netty/ForkJoin instrumentation prevents double-timing there.
66-
// stopTiming() is called in startTaskScope() when the task is activated on the worker.
67-
QueueTimerHelper.startQueuingTimer(state, null, null, 0, task);
68-
}
61+
state.captureAndSetContinuation(span);
6962
}
7063
}
7164
}

0 commit comments

Comments
 (0)