Don't suggest cache tags for Redis #9286
Merged
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.
The upgrade guide indicates that Redis cache tags got a performance improvement. However based on discussions linked below, it appears that usage of redis cache tags is not recommended. The outdated guidance in the upgrade guide caused a series of significant bugs in our production application. This PR updates the docs to recommend against using cache tags until a suitable solution can be found.
The current upgrade guide still contains a broken link to the documentation on cache tagging (which was removed as a result of Taylor's comment in the second link there).
If there is another recommended way for implementing Redis cache tags, I'd be happy to write up docs for that, but I'm not seeing one in my quick perusal of the issues, PRs, and conversations in github.