AdguardTeam / golibs

Small helper Go libraries
The Unlicense
18 stars 9 forks source link

Replacing Cache.lock with sync.RWMutex (instead of sync.Mutex) #4

Closed Eifoen closed 1 year ago

Eifoen commented 4 years ago

This should improve performance in excessive concurrent use cases

codecov-commenter commented 4 years ago

Codecov Report

Merging #4 into master will decrease coverage by 6.43%. The diff coverage is 51.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   83.43%   77.00%   -6.44%     
==========================================
  Files           6        6              
  Lines         163      187      +24     
==========================================
+ Hits          136      144       +8     
- Misses         17       33      +16     
  Partials       10       10              
Impacted Files Coverage Δ
cache/cache.go 100.00% <ø> (ø)
cache/cache_data.go 78.50% <51.42%> (-13.07%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3f698cd...0f0fbdf. Read the comment docs.