Remove infinite caches to prevent memory leak
What does this PR do? (Please give us a brief description of what this PR does.)
This PR resolves #600 (closed) and #601 (closed) by replacing all "infinite" cache lifetimes and cleaning up MemoryKVCache<T>
.
Expiration lifetimes of all memory / redis caches have been reset to a reasonable number and normalized between similar caches.
Redis caches have a synchronization fix, and an optimization is applied to MemoryKVCache.gc()
.
This is a smaller, more narrow-focused PR that replaces !580 (closed). A similar PR is open for Misskey: https://github.com/misskey-dev/misskey/pull/14363
Contribution Guidelines By submitting this merge request, you agree to follow our Contribution Guidelines
-
I agree to follow this project's Contribution Guidelines -
I have made sure to test this pull request
Merge request reports
Activity
mentioned in merge request !580 (closed)
- Resolved by dakkar
added bugbackend label
added bugfrontend label and removed bugbackend label
added enhancement label
removed enhancement label
- Resolved by Marie
Me waiting on the commit with the timeout comment fixes before approving
🙂
reset approvals from @dakkar by pushing to the branch
- Resolved by dakkar
- Resolved by Hazelnoot
ready to merge?
mentioned in commit 4e7df7a5
added bug::backend label and removed bug::frontend label