redlock 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,15 @@
 ---
-SHA1:
-  metadata.gz: e8e7d408cb9639c785875cbc975cd1fbb7166737
-  data.tar.gz: 4f067eef13f678f00b85d55edab568471e37b23e
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    N2ZjYTlmOTc2ZTc0MmYxZGE0MzBiNzY2ZjEzOGMxOWU4YjE3NzRhMQ==
+  data.tar.gz: !binary |-
+    NjMwOGMxNTRkMDliODFmYjI3YWY5MWQ3ZTdjMjUzMDNlNTdiNDQ4Yg==
 SHA512:
-  metadata.gz: 96f84d612fdd0ee0eb68b3d357d58e4f042077f10b5988d65fb95e01e2bb4b2f2758fb81671a002c29c76a361956e7207d36455ced2e37d25c0d14bbdf78501f
-  data.tar.gz: 7283eac7e5222740298f5cb0b9849397fdaeb196ed2f2a218311cd127544da93a2ac7fe7063d010500762c224a4d811ff88e7c48e05b859c697665d305af9ead
+  metadata.gz: !binary |-
+    ZTg2ZTI4MzY0ZmI3MzhkYzlhNTA3NzVjMGYyZmJlNGVhNjAzODE0ZGY4M2Qz
+    NDI2ODJmYzU5MGI2YjM3MmMxNGQwNTRlYzM4ZTE0MWNhNmQyYWVlNGUyYmU4
+    YWM0OTUyMzQwNzdlNTk2ZTNiNGE4OTdjYTQzYmYwMjBjYjU5YmE=
+  data.tar.gz: !binary |-
+    N2Q1ZTcwZWRlNDYyZjY0MzY4MDA4ZTVhMjEwNTE3N2Q4MWZmOTY0MDA0MmIx
+    MjVlYzM3ZWQ1Y2RkODJiOTI0ZTZiYWQ3MjdmZWU2NWZkNzMyMGM2YzM2OTUy
+    NWJmMjNmNjU0YzgyNDNkNGVhMDM2OWJjZmMzY2NhMWY4MzVjZjI=

data/README.md

@@ -84,6 +84,10 @@ end
 Make sure you have at least 1 redis instances up.
 
    $ rspec
+   
+## Disclaimer
+
+This code implements an algorithm which is currently a proposal, it was not formally analyzed. Make sure to understand how it works before using it in your production environments. You can see discussion about this approach at [reddit](http://www.reddit.com/r/programming/comments/2nt0nq/distributed_lock_using_redis_implemented_in_ruby/).
 
 ## Contributing
 

data/lib/redlock/client.rb

@@ -76,16 +76,12 @@ module Redlock
         end
       end
 
-      def redis
-        @redis
-      end
-
       def lock(resource, val, ttl)
-        redis.client.call([:set, resource, val, 'NX', 'PX', ttl])
+        @redis.set(resource, val, nx: true, px: ttl)
       end
 
       def unlock(resource, val)
-        redis.client.call([:eval, UNLOCK_SCRIPT, 1, resource, val])
+        @redis.eval(UNLOCK_SCRIPT, [resource], [val])
       rescue
         # Nothing to do, unlocking is just a best-effort attempt.
       end

data/lib/redlock/version.rb

@@ -1,3 +1,3 @@
 module Redlock
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/spec/client_spec.rb

@@ -12,10 +12,11 @@ RSpec.describe Redlock::Client do
       servers = [ 'redis://localhost:6379', Redis.new(url: 'redis://someotherhost:6379') ]
       redlock = Redlock::Client.new(servers)
 
-      redlock_servers = redlock.instance_variable_get(:@servers)
+      redlock_servers = redlock.instance_variable_get(:@servers).map do |s|
+        s.instance_variable_get(:@redis).client.host
+      end
 
-      expect(redlock_servers.one? { |s| s.redis.client.host == 'localhost' })
-      expect(redlock_servers.one? { |s| s.redis.client.port == 'someotherhost' })
+      expect(redlock_servers).to match_array(%w{ localhost someotherhost })
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: redlock
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Leandro Moreira
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-18 00:00:00.000000000 Z
+date: 2015-04-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -17,7 +17,7 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: '3'
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 3.0.5
   type: :runtime
@@ -27,7 +27,7 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: '3'
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 3.0.5
 - !ruby/object:Gem::Dependency
@@ -48,14 +48,14 @@ dependencies:
   name: coveralls
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
@@ -118,12 +118,12 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []