Skip to content
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

fix view cache singleflight #63

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

theharveyz
Copy link
Contributor

@theharveyz theharveyz commented Dec 18, 2017

RT
原有view_cache实现,未充分考虑到并发场景. 并发场景下, 当缓存过期时, 易引发缓存失效风暴

@codecov-io
Copy link

codecov-io commented Dec 18, 2017

Codecov Report

Merging #63 into master will decrease coverage by 0.27%.
The diff coverage is 86.36%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   69.49%   69.22%   -0.28%     
==========================================
  Files          43       40       -3     
  Lines        2203     2008     -195     
==========================================
- Hits         1531     1390     -141     
+ Misses        672      618      -54

@theharveyz theharveyz closed this Dec 19, 2017
@theharveyz theharveyz reopened this Dec 19, 2017
@theharveyz theharveyz closed this Dec 19, 2017
@theharveyz theharveyz reopened this Dec 20, 2017
@theharveyz theharveyz changed the title fix dog-pile effect fix view cache singleflight Jan 30, 2018
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.

None yet

2 participants