Skip to content

Conversation

@utpilla
Copy link
Contributor

@utpilla utpilla commented Mar 2, 2022

Fixes #2958

Changes

  • Add SpinWait for double Counter updates

@utpilla utpilla requested a review from a team March 2, 2022 01:56
@codecov
Copy link

codecov bot commented Mar 2, 2022

Codecov Report

Merging #2960 (7c46abf) into main (31475a5) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2960      +/-   ##
==========================================
- Coverage   84.07%   83.95%   -0.12%     
==========================================
  Files         255      255              
  Lines        9061     9063       +2     
==========================================
- Hits         7618     7609       -9     
- Misses       1443     1454      +11     
Impacted Files Coverage Δ
src/OpenTelemetry/Metrics/MetricPoint.cs 85.21% <100.00%> (+0.21%) ⬆️
...xporter.OpenTelemetryProtocol/OtlpTraceExporter.cs 36.36% <0.00%> (-22.73%) ⬇️
...entation/ExportClient/OtlpGrpcTraceExportClient.cs 35.71% <0.00%> (-14.29%) ⬇️
...mentation/ExportClient/BaseOtlpGrpcExportClient.cs 50.00% <0.00%> (-12.50%) ⬇️
...heus/Implementation/PrometheusCollectionManager.cs 79.74% <0.00%> (-2.54%) ⬇️

@cijothomas cijothomas merged commit 6973a53 into open-telemetry:main Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use SpinWait for double sum updates

3 participants