ActiveSession.cleaned_up_lookup_entries to use Redis#pipelined

9fd644f8 · Imre Farkas · Thong Kuah · f5fcb9a1 · 9fd644f8
Commit 9fd644f8 authored Aug 07, 2019 by Imre Farkas Committed by Thong Kuah Aug 07, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

app/models/active_session.rb app/models/active_session.rb +4 -2

No files found.
--- a/app/models/active_session.rb
+++ b/app/models/active_session.rb
@@ -136,8 +136,10 @@ class ActiveSession
    # only the single key entries are automatically expired by redis, the
    # lookup entries in the set need to be removed manually.
    session_ids_and_entries = session_ids.zip(entries)
-    session_ids_and_entries.reject { |_session_id, entry| entry }.each do |session_id, _entry|
+    redis.pipelined do
-      redis.srem(lookup_key_name(user.id), session_id)
+      session_ids_and_entries.reject { |_session_id, entry| entry }.each do |session_id, _entry|
+        redis.srem(lookup_key_name(user.id), session_id)
+      end
    end
    entries.compact