search

uber-go / tally

A Go metrics interface with fast buffered metrics and third party reporters
MIT License
850 stars 116 forks source link

fixing race condition in ForEachScope #212

Closed brawndou closed 1 year ago

brawndou commented 1 year ago

This fixes an issue in ForEachScope() where a subscopeBucket could be updated during snapshot.

codecov[bot] commented 1 year ago

Codecov Report

Merging #212 (5aca869) into master (007e844) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   47.81%   47.81%           
=======================================
  Files          64       64           
  Lines        5988     5988           
=======================================
  Hits         2863     2863           
  Misses       2685     2685           
  Partials      440      440
Impacted Files Coverage Δ
scope_registry.go 96.74% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more