README: Update the Caveats section #529
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 Caveats section of the README contained some stale information, which I removed. For example,
is no longer true after #471.
is no longer true after #242
I've also turned the brief descriptions of ways in which cache invalidations can be lost into a list. This should help a reader looking for the potential source of lost cache invalidations as well as for us to easily add to. I've also added 2 additional ways cache invalidations can be lost to the list that didn't seem to be documented, one that can occur during cache flushing and another that can occur during cache namespace changes.
I also wanted the caveats to explicitly discourage the use of Identity Cache for write requests.
Lastly, I wanted to document the problem with queuing background jobs with ids for uncommitted database records and how can lead to
nil
being cached for these records.