-
Notifications
You must be signed in to change notification settings - Fork 10
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
Wait for telemetry client to finish so the metrics are flushed. #862
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #862 +/- ##
==========================================
- Coverage 71.89% 71.89% -0.01%
==========================================
Files 322 322
Lines 48026 48043 +17
==========================================
+ Hits 34530 34539 +9
- Misses 13496 13504 +8
|
/// Shutdowns the telemetry client. | ||
pub async fn shutdown(&self) { | ||
pub async fn shutdown(self) { |
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.
❓
what would happen in case of start-shutdown-start?
we should add a test here for above.
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.
In that case the compiler will complain because the TelemetryClient
is consumed by the shutdown
method so there is no way to start again the client.
8404276
to
285d2f4
Compare
BenchmarksComparisonBenchmark execution time: 2025-02-07 10:14:21 Comparing candidate commit 285d2f4 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 52 metrics, 2 unstable metrics. CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
BaselineOmitted due to size. |
What does this PR do?
Waits for the task completion so the enqueued metrics are properly flushed.