paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-03 05:23:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

cache_helpers: Switch to storing the by-id user cache, not by-email.

Browse Source 
The email is not the most commonly used user cache -- the by-id one is.
This commit is contained in:
Alex Vandiver
2025-02-11 21:02:14 +00:00
committed by Tim Abbott
parent a7d513e5ec
commit 9ff68a45c5
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
zerver/lib/cache_helpers.py
View File

@@ -19,7 +19,7 @@ from zerver.lib.cache import (
get_remote_cache_requests,
get_remote_cache_time,
user_profile_by_api_key_cache_key,
user_profile_cache_key_id,
user_profile_by_id_cache_key,
)
from zerver.lib.safe_session_cached_db import SessionStore
from zerver.lib.sessions import session_engine
@@ -38,11 +38,7 @@ def user_cache_items(
items_for_remote_cache[user_profile_by_api_key_cache_key(user_profile.api_key)] = (
user_profile,
)
items_for_remote_cache[user_profile_cache_key_id(user_profile.email, user_profile.realm_id)] = (
user_profile,
)
# We have other user_profile caches, but none of them are on the
# core serving path for lots of requests.
items_for_remote_cache[user_profile_by_id_cache_key(user_profile.id)] = (user_profile,)
def client_cache_items(items_for_remote_cache: dict[str, tuple[Client]], client: Client) -> None: