search

open-telemetry / opentelemetry-cpp

The OpenTelemetry C++ Client
https://opentelemetry.io/
Apache License 2.0
817 stars 392 forks source link

[WIP] ForceFlush before canceling the running requests on shutdown. #2727

Open owent opened 6 days ago

owent commented 6 days ago

Fixes #2715

Changes

Sorry for having no time to test yet, I will test it next week when I have time

For significant contributions please make sure you have completed the following items:

codecov[bot] commented 6 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.66%. Comparing base (497eaf4) to head (cb20569). Report is 96 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/2727/graphs/tree.svg?width=650&height=150&src=pr&token=FJESTYQ2AD&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry)](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/2727?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry) ```diff @@ Coverage Diff @@ ## main #2727 +/- ## ========================================== + Coverage 87.12% 87.66% +0.54% ========================================== Files 200 190 -10 Lines 6109 5855 -254 ========================================== - Hits 5322 5132 -190 + Misses 787 723 -64 ``` [see 110 files with indirect coverage changes](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/2727/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry)