8000 Defer remote components to avoid OkHttp class-loading side-effects by mcculls · Pull Request #8131 · DataDog/dd-trace-java · GitHub
[go: up one dir, main page]
Skip to content

Defer remote components to avoid OkHttp class-loading side-effects #8131

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 8 commits into from
Dec 30, 2024
Merged

Defer remote components to avoid OkHttp class-loading side-effects #8131

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
92ca151
Additional logging to help triage custom log-manager / jmx-builder te…
mcculls Dec 28, 2024
40c7583
Move SharedCommunicationObjects creation to InstallDatadogTracerCallb…
mcculls Dec 22, 2024
84d2b6d
Add ability to pause remote components until SharedCommunicationObjec…
mcculls Dec 28, 2024
9b97d7a
Delay starting trace-writer and data-stream-monitoring until remote I…
mcculls Dec 28, 2024
2cdc507
Delay starting logs-intake backend until remote I/O is allowed
mcculls Dec 28, 2024
9a09d98
Install tracer and logs-intake as soon as necessary, but defer any re…
mcculls Dec 28, 2024
9ee929b
Remove exemption where we didn't defer if the custom logging manager
mcculls Dec 28, 2024
c9cf18f
Adjust minimumBranchCoverage to account for new double-checked lock i…
mcculls Dec 28, 2024
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
Prev Previous commit
Next Next commit
Delay starting trace-writer and data-stream-monitoring until remote I… 
…/O is allowed
  • Loading branch information
@mcculls
mcculls committed Dec 28, 2024
commit 9b97d7af2f827142e0d6c57952560f90659f4ff3
5 changes: 3 additions & 2 deletions dd-trace-core/src/main/java/datadog/trace/core/CoreTracer.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -689,7 +689,7 @@ private CoreTracer(
}
pendingTraceBuffer.start();

this.writer.start();
sharedCommunicationObjects.whenReady(this.writer::start);

metricsAggregator = createMetricsAggregator(config, sharedCommunicationObjects);
// Schedule the metrics aggregator to begin reporting after a random delay of 1 to 10 seconds
Expand All @@ -705,7 +705,8 @@ private CoreTracer(
} else {
this.dataStreamsMonitoring = dataStreamsMonitoring;
}
this.dataStreamsMonitoring.start();

sharedCommunicationObjects.whenReady(this.dataStreamsMonitoring::start);

// Create default extractor from config if not provided and decorate it with DSM extractor
HttpCodec.Extractor builtExtractor =
Expand Down
0