minor cleanup to previous commit from code review

2025-05-22 03:21:12 +08:00 · 2018-09-19 16:07:29 +10:00
parent 5302709343
commit e0e6dae6a7
2 changed files with 6 additions and 4 deletions
--- a/lib/distributed_mutex.rb
+++ b/lib/distributed_mutex.rb
@ -22,10 +22,12 @@ class DistributedMutex

    while !try_to_get_lock
      sleep 0.001
-      attempts += 1
      # in readonly we will never be able to get a lock
-      if @using_global_redis && attempts > CHECK_READONLY_ATTEMPT
-        raise Discourse::ReadOnly
+      if @using_global_redis
+        attempts += 1
+        if attempts > CHECK_READONLY_ATTEMPT
+          raise Discourse::ReadOnly
+        end
      end
    end