Cache user latest packs query

MTG · Oct 31, 2023 · 44a4ce1 · 44a4ce1
1 parent fe54200
commit 44a4ce1
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/accounts/views.py b/accounts/views.py
@@ -25,6 +25,7 @@
 import json
 import logging
 import os
+import pickle
 import tempfile
 import time
 import uuid
@@ -1485,9 +1486,14 @@ def account_latest_packs_section(request, username):
     if not request.GET.get('ajax'):
         raise Http404  # Only accessible via ajax
     user = request.parameter_user
-    latest_pack_ids = Pack.objects.select_related().filter(user=user, num_sounds__gt=0).exclude(is_deleted=True) \
-                        .order_by("-last_updated").values_list('id', flat=True)[0:15]
-    latest_packs = Pack.objects.ordered_ids(pack_ids=latest_pack_ids)
+    latest_packs_from_cache = cache.get(settings.USER_LATEST_PACKS_CACHE_KEY.format(user.id), None)
+    if latest_packs_from_cache is None:
+        latest_pack_ids = Pack.objects.select_related().filter(user=user, num_sounds__gt=0).exclude(is_deleted=True) \
+                            .order_by("-last_updated").values_list('id', flat=True)[0:15]
+        latest_packs = Pack.objects.ordered_ids(pack_ids=latest_pack_ids)
+        cache.set(settings.USER_LATEST_PACKS_CACHE_KEY.format(user.id), pickle.dumps(latest_packs), 60 * 60 * 24)
+    else:
+        latest_packs = pickle.loads(latest_packs_from_cache)
     tvars = {
         'user': user,
         'latest_packs': latest_packs,

diff --git a/freesound/settings.py b/freesound/settings.py
@@ -351,8 +351,9 @@
 # Whether or not user bookmarks and bookmark categories should be public (visible to all other users) in BW
 BW_BOOKMARK_PAGES_PUBLIC = False
 
-# User stats cache key template
+# User profile page cache key templates
 USER_STATS_CACHE_KEY = 'user_stats_{}'
+USER_LATEST_PACKS_CACHE_KEY = 'user_latest_packs_{}'
 
 # User flagging notification thresholds
 USERFLAG_THRESHOLD_FOR_NOTIFICATION = 3

diff --git a/utils/cache.py b/utils/cache.py
@@ -41,6 +41,7 @@ def invalidate_user_template_caches(user_id):
     invalidate_template_cache('bw_user_profile_following_count', user_id)
     invalidate_template_cache('bw_user_profile_following_tags_count', user_id)
     cache.delete(settings.USER_STATS_CACHE_KEY.format(user_id))
+    cache.delete(settings.USER_LATEST_PACKS_CACHE_KEY.format(user_id))
 
 
 def invalidate_all_moderators_header_cache():