mlanett-redis-lock 0.1.0 → 0.1.1

data/lib/redis-lock/version.rb

@@ -1,5 +1,5 @@
 class Redis
   class Lock
-    VERSION = "0.1.0"
+    VERSION = "0.1.1"
   end
 end

data/lib/redis-lock.rb

@@ -162,7 +162,7 @@ class Redis
       return false
     end
 
-    # Calls block until it returns true or times out. Uses exponential backoff.
+    # Calls block until it returns true or times out. TODO: Use exponential backoff.
     # @param block should return true if successful, false otherwise
     # @returns true if successful, false otherwise
     def with_timeout( timeout, &block )
@@ -173,7 +173,8 @@ class Redis
         return true if block.call
         log :debug, "Timeout" and return false if Time.now + sleepy > expire
         sleep(sleepy)
-        sleepy *= 2
+        # might like a different strategy, but general goal is not use 100% cpu while contending for a lock.
+        # sleepy = [ sleepy * 2, ( expire - Time.now ) / 4 ].min
       end
     end
 

data/spec/redis_lock_spec.rb

@@ -55,7 +55,7 @@ describe Redis::Lock, redis: true do
     results = [ false, false, true ]
     hers.with_timeout(1) { results.shift }.should be_true
     # this is too many attemps
-    results = [ false, false, false, false, false, true ]
+    results = [ false, false, false, false, false, false, false, false, false, false, true ]
     hers.with_timeout(1) { results.shift }.should be_false
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mlanett-redis-lock
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-07 00:00:00.000000000Z
+date: 2012-03-14 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
-  requirement: &70349318263680 !ruby/object:Gem::Requirement
+  requirement: &70189385787000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +21,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70349318263680
+  version_requirements: *70189385787000
 description: Pessimistic locking using Redis
 email:
 - mark.lanett@gmail.com
@@ -64,7 +64,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.15
+rubygems_version: 1.8.16
 signing_key: 
 specification_version: 3
 summary: Pessimistic locking using Redis