-
Notifications
You must be signed in to change notification settings - Fork 775
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
Add otlp log and trace exporter benchmarks #4807
Add otlp log and trace exporter benchmarks #4807
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4807 +/- ##
=======================================
Coverage 83.54% 83.54%
=======================================
Files 292 292
Lines 11987 11987
=======================================
Hits 10014 10014
Misses 1973 1973 |
…vishweshbankwar/opentelemetry-dotnet into vibankwa/add-benchmark-log-otlp
I have a general comment which doesn't have to block this PR: how do we consider the granularity of the Benchmark project? (take Stress Test for example, we don't shove all components' test logic into a single test project) |
@reyang We do not have a set rule for deciding granularity. Not sure if we have a need for individual projects for benchmarks as its easier to use filtering during execution. |
Co-authored-by: Utkarsh Umesan Pillai <[email protected]>
Co-authored-by: Utkarsh Umesan Pillai <[email protected]>
…vishweshbankwar/opentelemetry-dotnet into vibankwa/add-benchmark-log-otlp
Fixes #
Design discussion issue #
Changes
Please provide a brief description of the changes here.
Merge requirement checklist
CHANGELOG.md
files updated for non-trivial changes