Merge pull request #5 from keylimetoolbox/double-initial-set

Fix #initial_set which is causing a double attempt and delay on lock acquisition and incorrect drop on short acquisition_timeout

lib/suo/client/base.rb

@@ -56,10 +56,7 @@ def refresh(token)
         retry_with_timeout do
           val, cas = get
 
-          if val.nil?
-            initial_set
-            next
-          end
+          cas = initial_set if val.nil?
 
           cleared_locks = deserialize_and_clear_locks(val)
 
@@ -102,10 +99,7 @@ def acquire_lock(token = nil)
         retry_with_timeout do
           val, cas = get
 
-          if val.nil?
-            initial_set
-            next
-          end
+          cas = initial_set if val.nil?
 
           cleared_locks = deserialize_and_clear_locks(val)
 

lib/suo/client/memcached.rb

@@ -26,6 +26,8 @@ def set(newval, cas, expire: false)
 
       def initial_set(val = BLANK_STR)
         @client.set(@key, val)
+        _val, cas = @client.get_cas(@key)
+        cas
       end
     end
   end

lib/suo/client/redis.rb

@@ -39,7 +39,8 @@ def synchronize
       end
 
       def initial_set(val = BLANK_STR)
-        @client.set(@key, val)
+        set(val, nil)
+        nil
       end
     end
   end