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

prevent multiple requests while filling the cache #113

Closed
wants to merge 1 commit into from

Conversation

zerotacg
Copy link

while proxy_cache_lock_timeout is set to ensure the request is cached, there still can happen multiple requests to the origin server while the entry is being cached. This would result in reduced bandwidth for the proxy itself.

[0] https://nginx.org/en/docs/http/ngx_http_proxy_module.html#proxy_cache_lock_age

VibroAxe added a commit to lancachenet/monolithic that referenced this pull request Oct 9, 2021
This allows us to cache the response in the event a file hasn't been downloaded yet
Set to 2m as a really long time to download a single 1m slice from upstream

Based on thoughts from lancachenet/generic#113
VibroAxe added a commit to lancachenet/monolithic that referenced this pull request Oct 9, 2021
@VibroAxe
Copy link
Member

VibroAxe commented Oct 9, 2021

Updated and Merged via lancachenet/monolithic#133

@VibroAxe VibroAxe closed this Oct 9, 2021
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.

2 participants