Simplify ForwardingCacheBackend to reduce state storage.
Review Request #13449 — Created Dec. 5, 2023 and submitted
When
ForwardingCacheBackend
was first created, it had to work around
some assumptions in Django where a cache backend could be created but
then never replaced.Modern Django manages cache backends through a mutable registry,
removing much of the need forForwardingCacheBackend
. However, it's
still a useful API.In an effort to keep the logic simple, and to remove any
threading-related requirements in the implementation, this change
updates the backend to just look up the named backend on demand, rather
than storing an instance of it. This should avoid any issues where a
stale backend could ever be in use, and allows Django's connection
management to handle any threading responsibility.There's also a fix in the code for applying siteconfig settings. The old
code assumed thatdjango.core.cache.cache
was an instance of the
backend, but it's not anymore. So instead, we now optimistically call
the function we need on it, ignoring any attribute errors if not found.
All Djblets and Review Board unit tests pass.
Tested this with a local server. Verified that when I changed settings,
the cache updated.
Summary | ID |
---|---|
bdd536f12f01356f8f5abf2f0b9a590277e8df4d |