diff options
| author | Neil Alexander <neilalexander@users.noreply.github.com> | 2022-08-03 17:14:21 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-08-03 17:14:21 +0100 |
| commit | 2250768be16bd0e6b3a6a72b5e55eb3e2ad6e3c6 (patch) | |
| tree | 71b0429816fec0b74895e1907d3c32923400fde0 /internal/caching/cache_roominfo.go | |
| parent | bbff41b44bff2dbc53867cc0fd94ce8f31fd511a (diff) | |
Remove roominfo cache (#2615)
* Remove roominfo cache
It's the source of a number of race conditions which are seemingly causing bugs and CI failures.
* Make the linter less sad
Diffstat (limited to 'internal/caching/cache_roominfo.go')
| -rw-r--r-- | internal/caching/cache_roominfo.go | 33 |
1 files changed, 0 insertions, 33 deletions
diff --git a/internal/caching/cache_roominfo.go b/internal/caching/cache_roominfo.go deleted file mode 100644 index 5dfed3c8..00000000 --- a/internal/caching/cache_roominfo.go +++ /dev/null @@ -1,33 +0,0 @@ -package caching - -import ( - "github.com/matrix-org/dendrite/roomserver/types" -) - -// WARNING: This cache is mutable because it's entirely possible that -// the IsStub or StateSnaphotNID fields can change, even though the -// room version and room NID fields will not. This is only safe because -// the RoomInfoCache is used ONLY within the roomserver and because it -// will be kept up-to-date by the latest events updater. It MUST NOT be -// used from other components as we currently have no way to invalidate -// the cache in downstream components. - -// RoomInfosCache contains the subset of functions needed for -// a room Info cache. It must only be used from the roomserver only -// It is not safe for use from other components. -type RoomInfoCache interface { - GetRoomInfo(roomID string) (roomInfo *types.RoomInfo, ok bool) - StoreRoomInfo(roomID string, roomInfo *types.RoomInfo) -} - -// GetRoomInfo must only be called from the roomserver only. It is not -// safe for use from other components. -func (c Caches) GetRoomInfo(roomID string) (*types.RoomInfo, bool) { - return c.RoomInfos.Get(roomID) -} - -// StoreRoomInfo must only be called from the roomserver only. It is not -// safe for use from other components. -func (c Caches) StoreRoomInfo(roomID string, roomInfo *types.RoomInfo) { - c.RoomInfos.Set(roomID, roomInfo) -} |