[Workers] Expliclity call out Cache Reserve incompatibility - WIP - DO NOT MERGE #17017
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
markdembo
commented
Sep 23, 2024
markdembo
commented
- Explicitly state that Cache Reserve is not supported in Workers docs
markdembo
requested review from
a team,
GregBrimble,
WalshyDev,
irvinebroque,
kodster28 and
mikenomitch
as code owners
github-actions
bot
assigned
GregBrimble,
irvinebroque,
kodster28,
mikenomitch and
WalshyDev
Deploying cloudflare-docs with
|
| Latest commit: |
145e28e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://61ea8c76.cloudflare-docs-7ou.pages.dev |
| Branch Preview URL: | https://mdembo-workerscachereserve.cloudflare-docs-7ou.pages.dev |
| :::caution[Tiered caching and the Cache API] | ||
|
|
||
| Cache API within Workers does not support tiered caching. Tiered Cache concentrates connections to origin servers so they come from a small number of data centers rather than the full set of network locations. Cache API is local to a data center, this means that `cache.match` does a lookup, `cache.put` stores a response, and `cache.delete` removes a stored response only in the cache of the data center that the Worker handling the request is in. Because these methods apply only to local cache, they will not work with tiered cache. | ||
| Cache API within Workers does not support tiered caching or use with Cache Reserve. Tiered Cache concentrates connections to origin servers so they come from a small number of data centers rather than the full set of network locations. Cache API is local to a data center, this means that `cache.match` does a lookup, `cache.put` stores a response, and `cache.delete` removes a stored response only in the cache of the data center that the Worker handling the request is in. Because these methods apply only to local cache, they will not work with tiered cache. |
Contributor
There was a problem hiding this comment.
This should tell people to use KV
Contributor
markdembo
changed the title
dcpena
reviewed
Sep 27, 2024
|
|
||
| ## Using Cache API | ||
|
|
||
| The Cache API allows for a simple API to cache POST requests. However, Cache API does not [support the use of Tiered Caching or Reserve Reserve](/workers/reference/how-the-cache-works/#cache-api). To increase Cache Hit Rates and offload origin load, consider using Workers KV instead (see below). |
Collaborator
There was a problem hiding this comment.
Suggested change
| The Cache API allows for a simple API to cache POST requests. However, Cache API does not [support the use of Tiered Caching or Reserve Reserve](/workers/reference/how-the-cache-works/#cache-api). To increase Cache Hit Rates and offload origin load, consider using Workers KV instead (see below). | |
| The Cache API allows for a simple API to cache POST requests. However, Cache API does not support the use of [Tiered Caching or Reserve Reserve](/workers/reference/how-the-cache-works/#cache-api). To increase Cache Hit Rates and offload origin load, consider using Workers KV instead (see below). |
Collaborator
There was a problem hiding this comment.
I suggested which part of the text was included as the link to prevent potential confusion.
|
|
||
| ## Using Workers KV | ||
|
|
||
| KV allows |
Collaborator
There was a problem hiding this comment.
I know this PR is still a WIP so may get addressed later but this reads as incomplete for now.
Collaborator
|
Closing this out b/c there's conflicts + hasn't been touched for quite a while. feel free to re-open / bug me if this is in error. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.