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

Increase block size for segmented caching #345

Merged
merged 3 commits into from
Oct 3, 2023

Conversation

jdno
Copy link
Member

@jdno jdno commented Sep 11, 2023

Increasing the block size for segmented caching results in much better performance, but might cause segments to be evicted from the cache earlier and thus lowering the cache hit rate. Fastly does not recommend raising the block size above 10MB for this exact reason. We are starting at this limit to optimize for performance, and might lower the size if we observe issues with the cache hit rate.

Read more: https://developer.fastly.com/reference/vcl/variables/segmented-caching/segmented-caching-block-size/

This change also enables the streaming miss feature, which improves the time to first-byte. We've enabled it while investigating the slow download speeds.

Read more: https://docs.fastly.com/en/guides/streaming-miss

The streaming miss[^1] feature improves the time to first-byte. We've
enabled it while investigating slow download speeds for uncached
artifacts from Europe.

[^1]: https://docs.fastly.com/en/guides/streaming-miss
Increasing the block size for segmented caching results in much better
performance, but might cause segments to be evicted from the cache
earlier and thus lowering the cache hit rate. Fastly does not recommend
raising the block size above 10MB for this exact reason. We are starting
at this limit to optimize for performance, and might lower the size if
we observe issues with the cache hit rate.

Read more: https://developer.fastly.com/reference/vcl/variables/segmented-caching/segmented-caching-block-size/
@jdno
Copy link
Member Author

jdno commented Sep 11, 2023

I've deployed this to both staging and production and am watching the metrics closely to ensure everything works smoothly. The change in block sizes busted the cache, but the cache hit rate is slowly returning to normal levels.

@meysam81
Copy link
Contributor

Is there any public Grafana to see the result of this change? 🤔

@jdno
Copy link
Member Author

jdno commented Oct 3, 2023

Is there any public Grafana to see the result of this change? 🤔

No. I checked this in Fastly's dashboard. We theoretically send the metrics to Prometheus as well, but we haven't created a dashboard for it yet.

@jdno jdno merged commit 1714902 into rust-lang:master Oct 3, 2023
@jdno jdno deleted the segmented-caching-block-size branch October 3, 2023 10:09
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.

3 participants