opentelemetry-dotnet icon indicating copy to clipboard operation
opentelemetry-dotnet copied to clipboard

Metric - concurrency test improvements

Open cijothomas opened this issue 4 years ago • 1 comments

Current tests are very limited: https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/test/OpenTelemetry.Tests/Metrics/MetricAPITest.cs#L241

  1. Need to test the special case of "no tags".
  2. Need to add test where different tags are updated by diff. threads. (To catch these https://github.com/open-telemetry/opentelemetry-dotnet/pull/2339#discussion_r711239998)
  3. Need to add test where same tags are updated by diff. threads. (already covered for single dimension. Need to modify for more dimensions as there are special ThreadLocal optimizations for upto N.)

cijothomas avatar Sep 17 '21 17:09 cijothomas

This issue was marked stale due to lack of activity and will be closed in 7 days. Commenting will instruct the bot to automatically remove the label. This bot runs once per day.

github-actions[bot] avatar Aug 27 '24 03:08 github-actions[bot]

Closed as inactive. Feel free to reopen if this issue is still a concern.

github-actions[bot] avatar Sep 03 '24 03:09 github-actions[bot]