distlock 0.0.8 → 0.0.9

data/lib/distlock.rb

@@ -1,11 +1,26 @@
+require 'logger'
 require 'distlock/version'
 require 'distlock/lock_error'
 require 'distlock/zk/zk'
 
 module Distlock
+  def Distlock.logger
+    @logger ||= Logger.new(STDOUT)
+  end
+
+  def Distlock.logger=(logger)
+    @logger = logger
+  end
+
+  # factory method for creating instances of lock managers
+  def Distlock.new_instance(lock_type = :zk_exclusive_lock, options={})
+    locker = Distlock::ZK::ExclusiveLock.new(options)
+    locker.logger = Distlock.logger
+    locker
+  end
 end
 
 # convenience so we can use either Distlock or DistLock
 module DistLock
   include Distlock
-end
+end

data/lib/distlock/version.rb

@@ -1,3 +1,3 @@
 module Distlock
-  VERSION = "0.0.8"
+  VERSION = "0.0.9"
 end

data/lib/distlock/zk/exclusive_lock.rb

@@ -49,6 +49,7 @@ module Distlock
       end
 
       def lock(path)
+        raise LockError.new("invalid lock path, must start with '/'") unless path.start_with?("/")
         @owner = false
         
         safe_create(path)
@@ -85,8 +86,13 @@ module Distlock
 
           path = lock.split('/')[0...-1].join('/')
         
-          # TODO - pass this in as parameter?
+          # TODO - pass children in as parameter?
           children = zk.get_children(:path => path)[:children].sort{|a,b|a.split('-').last <=> b.split('-').last}
+          
+          puts lock
+          puts path
+          puts children.inspect
+
           lock_last = lock.split('/').last
           lock_idx = children.index(lock_last)
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: distlock
 version: !ruby/object:Gem::Version 
-  hash: 15
+  hash: 13
   prerelease: false
   segments: 
   - 0
   - 0
-  - 8
-  version: 0.0.8
+  - 9
+  version: 0.0.9
 platform: ruby
 authors: 
 - Simon Horne
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-04-11 00:00:00 -04:00
+date: 2012-04-13 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency