gush 0.3.3 → 0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d26693f019dfd49b57cb7828274bec009c96dfb4
-  data.tar.gz: 1666a4d1edc877d4ac597428ed56f73a5f625fa9
+  metadata.gz: 6742c4a3bbcd13c0045c12c8733dae7e7b2fd2a2
+  data.tar.gz: 43293c3a05639fc46e3d39b87e750358a3ed4c60
 SHA512:
-  metadata.gz: b1f55d88bbe1719395103c6eeeda6a9d140d9cd7b6d35296f1ab97b1d381a7f69bbbe83e21faa660a05fe6183d7d5443c006244cbbcc99ebbc58edf554d2adbf
-  data.tar.gz: c2d6e5aa2a0613dae3e893ca368e527cac28fe1b9d6df66793de83bfe73dcad7940ee9d0cfd0663e8fd1fcc30eb313c0e465f98da1a350f5958d82beef9faa61
+  metadata.gz: 36263afadc504f4051a93613b27ab18f77b79b24c7e25927d11fc92322b5c42c4054fcd3d8dc2335843dc59224a1aab699a848459ce879b31920b5393d16c0f4
+  data.tar.gz: b4447f6c90b7ef427624eb6bdd3f2b1eb36a954b92f1c578dec3a446cbe7e7ea909d02957050e3779902db88e9cb4a12695bc5dc0f08432813d8925d37b7f92a

data/CHANGELOG.md

@@ -1,3 +1,3 @@
-# 0.4
+# 0.5
 
 - remove hard dependency on Yajl, so Gush can work with non-MRI Rubies ([#31](https://github.com/chaps-io/gush/pull/31) by [Nick Rakochy](https://github.com/chaps-io/gush/pull/31))

data/Gemfile

@@ -4,3 +4,4 @@ gemspec
 platforms :mri, :ruby do
   gem 'yajl-ruby'
 end
+

data/gush.gemspec

@@ -4,7 +4,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = "gush"
-  spec.version       = "0.3.3"
+  spec.version       = "0.4"
   spec.authors       = ["Piotrek Okoński"]
   spec.email         = ["piotrek@okonski.org"]
   spec.summary       = "Fast and distributed workflow runner using only Sidekiq and Redis"

data/lib/gush/client.rb

@@ -1,15 +1,17 @@
 module Gush
   class Client
-    attr_reader :configuration, :sidekiq
+    attr_reader :configuration
 
     def initialize(config = Gush.configuration)
       @configuration = config
       @sidekiq = build_sidekiq
+      @redis = build_redis
     end
 
     def configure
       yield configuration
       @sidekiq = build_sidekiq
+      @redis = build_redis
     end
 
     def create_workflow(name)
@@ -47,11 +49,7 @@ module Gush
       loop do
         id = SecureRandom.uuid
         job_identifier = "#{job_klass}-#{id}"
-        available = connection_pool.with do |redis|
-          !redis.exists("gush.jobs.#{workflow_id}.#{job_identifier}")
-        end
-
-        break if available
+        break if !redis.exists("gush.jobs.#{workflow_id}.#{job_identifier}")
       end
 
       job_identifier
@@ -61,54 +59,40 @@ module Gush
       id = nil
       loop do
         id = SecureRandom.uuid
-        available = connection_pool.with do |redis|
-          !redis.exists("gush.workflow.#{id}")
-        end
-
-        break if available
+        break if !redis.exists("gush.workflow.#{id}")
       end
 
       id
     end
 
     def all_workflows
-      connection_pool.with do |redis|
-        redis.keys("gush.workflows.*").map do |key|
-          id = key.sub("gush.workflows.", "")
-          find_workflow(id)
-        end
+      redis.keys("gush.workflows.*").map do |key|
+        id = key.sub("gush.workflows.", "")
+        find_workflow(id)
       end
     end
 
     def find_workflow(id)
-      connection_pool.with do |redis|
-        data = redis.get("gush.workflows.#{id}")
-
-        unless data.nil?
-          hash = Gush::JSON.decode(data, symbolize_keys: true)
-          keys = redis.keys("gush.jobs.#{id}.*")
-          nodes = redis.mget(*keys).map { |json| Gush::JSON.decode(json, symbolize_keys: true) }
-          workflow_from_hash(hash, nodes)
-        else
-          raise WorkflowNotFound.new("Workflow with given id doesn't exist")
-        end
+      data = redis.get("gush.workflows.#{id}")
+      unless data.nil?
+        hash = Gush::JSON.decode(data, symbolize_keys: true)
+        keys = redis.keys("gush.jobs.#{id}.*")
+        nodes = redis.mget(*keys).map { |json| Gush::JSON.decode(json, symbolize_keys: true) }
+        workflow_from_hash(hash, nodes)
+      else
+        raise WorkflowNotFound.new("Workflow with given id doesn't exist")
       end
     end
 
     def persist_workflow(workflow)
-      connection_pool.with do |redis|
-        redis.set("gush.workflows.#{workflow.id}", workflow.to_json)
-      end
-
+      redis.set("gush.workflows.#{workflow.id}", workflow.to_json)
       workflow.jobs.each {|job| persist_job(workflow.id, job) }
       workflow.mark_as_persisted
       true
     end
 
     def persist_job(workflow_id, job)
-      connection_pool.with do |redis|
-        redis.set("gush.jobs.#{workflow_id}.#{job.name}", job.to_json)
-      end
+      redis.set("gush.jobs.#{workflow_id}.#{job.name}", job.to_json)
     end
 
     def load_job(workflow_id, job_id)
@@ -116,16 +100,10 @@ module Gush
       job_name_match = /(?<klass>\w*[^-])-(?<identifier>.*)/.match(job_id)
       hypen = '-' if job_name_match.nil?
 
-      keys = connection_pool.with do |redis|
-        redis.keys("gush.jobs.#{workflow_id}.#{job_id}#{hypen}*")
-      end
-
+      keys = redis.keys("gush.jobs.#{workflow_id}.#{job_id}#{hypen}*")
       return nil if keys.nil?
 
-      data = connection_pool.with do |redis|
-        redis.get(keys.first)
-      end
-
+      data = redis.get(keys.first)
       return nil if data.nil?
 
       data = Gush::JSON.decode(data, symbolize_keys: true)
@@ -133,16 +111,12 @@ module Gush
     end
 
     def destroy_workflow(workflow)
-      connection_pool.with do |redis|
-        redis.del("gush.workflows.#{workflow.id}")
-      end
+      redis.del("gush.workflows.#{workflow.id}")
       workflow.jobs.each {|job| destroy_job(workflow.id, job) }
     end
 
     def destroy_job(workflow_id, job)
-      connection_pool.with do |redis|
-        redis.del("gush.jobs.#{workflow_id}.#{job.name}")
-      end
+      redis.del("gush.jobs.#{workflow_id}.#{job.name}")
     end
 
     def worker_report(message)
@@ -166,6 +140,8 @@ module Gush
 
     private
 
+    attr_reader :sidekiq, :redis
+
     def workflow_from_hash(hash, nodes = nil)
       flow = hash[:klass].constantize.new *hash[:arguments]
       flow.jobs = []
@@ -180,9 +156,7 @@ module Gush
     end
 
     def report(key, message)
-      connection_pool.with do |redis|
-        redis.publish(key, Gush::JSON.encode(message))
-      end
+      redis.publish(key, Gush::JSON.encode(message))
     end
 
 
@@ -195,7 +169,7 @@ module Gush
     end
 
     def connection_pool
-      @connection_pool ||= ConnectionPool.new(size: configuration.concurrency, timeout: 1) { build_redis }
+      ConnectionPool.new(size: configuration.concurrency, timeout: 1) { build_redis }
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gush
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: '0.4'
 platform: ruby
 authors:
 - Piotrek Okoński
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-23 00:00:00.000000000 Z
+date: 2016-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sidekiq
@@ -268,7 +268,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Fast and distributed workflow runner using only Sidekiq and Redis