taskinator 0.3.9 → 0.3.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4f8f0c88f1bf0ade876afc82c451b310064d0e07
-  data.tar.gz: 920e74b55fef2506b49e9a2ec21389cf646ef72d
+  metadata.gz: 1cac7446426c45fc14b6cc0dee7620874e322fa2
+  data.tar.gz: 7dd667765ef5e6a139b99ea03604aef198f80130
 SHA512:
-  metadata.gz: dd62e9056ce98cf2e78aa51dd95a3ce24d14e0fb894bfc699afca93950bca9c54d9c39a9259dc180a10ca7ac97a1bf76fa133cd38058833e38a5ee114920d0e1
-  data.tar.gz: 9dec75c348b5a66f55526b15fe7f04e76992e574d066667134bab276c27694ae60a29bce396b1aa668ef367b6ec491ef8276034984dca769f782096ab6681a6f
+  metadata.gz: de3cc2b3dd447ed10b1662cca01ea0398c35c375466ee3a55b3ed67eefc28cd1a8d5888e56f50258bc2027f7ea4f8462ba0bc52cac2d2f242dbd56b46b7548b4
+  data.tar.gz: c98d31d5968ecfc7289831b327a1fd00bb9be75c0206304f3d8aed6b38fbf3ec4d3460b85b3cf439604ea2490095d8fee60280c27c43a18b71c6a9b3aa47d03c

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    taskinator (0.3.9)
+    taskinator (0.3.10)
       builder (>= 3.2.2)
       connection_pool (>= 2.2.0)
       globalid (~> 0.3)

data/lib/taskinator/persistence.rb

@@ -68,6 +68,15 @@ module Taskinator
         end
       end
 
+      def to_xml
+        builder = ::Builder::XmlMarkup.new
+        builder.instruct!
+        builder.tag!('process', :key => self.key) do |xml|
+          XmlSerializationVisitor.new(xml, self).visit
+        end
+        builder
+      end
+
       # the persistence key
       def key
         @key ||= self.class.key_for(self.uuid)
@@ -262,7 +271,10 @@ module Taskinator
         tasks.each do |task|
           RedisSerializationVisitor.new(@conn, task, @base_visitor).visit
           @conn.rpush "#{@key}:tasks", task.uuid
-          @base_visitor.incr_task_count unless task.is_a?(Task::SubProcess)
+          unless task.is_a?(Task::SubProcess)
+            incr_task_count unless self == @base_visitor
+            @base_visitor.incr_task_count
+          end
         end
       end
 
@@ -315,6 +327,118 @@ module Taskinator
       end
     end
 
+    class XmlSerializationVisitor < Taskinator::Visitor::Base
+
+      #
+      # the redis connection is passed in since it is
+      # in the multi statement mode in order to produce
+      # one roundtrip to the redis server
+      #
+
+      attr_reader :builder
+      attr_reader :instance
+
+      def initialize(builder, instance, base_visitor=self)
+        @builder      = builder
+        @instance     = instance
+        @key          = instance.key
+        @root         = base_visitor.instance
+        @base_visitor = base_visitor
+        @task_count   = 0
+      end
+
+      # the starting point for serializing the instance
+      def visit
+        @attributes = []
+        @attributes << [:type, @instance.class.name]
+        @attributes << [:process_uuid, @root.uuid]
+        @attributes << [:state, :initial]
+
+        @instance.accept(self)
+
+        @attributes << [:task_count, @task_count]
+
+        @attributes.each do |(name, value)|
+          builder.tag!('attribute', name => value)
+        end
+
+        self
+      end
+
+      def visit_process(attribute)
+        process = @instance.send(attribute)
+        if process
+          @attributes << [attribute, process.uuid]
+
+          builder.tag!('process', :key => process.key) do |xml|
+            XmlSerializationVisitor.new(xml, process, @base_visitor).visit
+          end
+        end
+      end
+
+      def visit_tasks(tasks)
+        builder.tag!('tasks') do |xml|
+          tasks.each do |task|
+            xml.tag!('task', :key => task.key) do |xml2|
+              XmlSerializationVisitor.new(xml2, task, @base_visitor).visit
+              unless task.is_a?(Task::SubProcess)
+                incr_task_count unless self == @base_visitor
+                @base_visitor.incr_task_count
+              end
+            end
+          end
+        end
+      end
+
+      def visit_attribute(attribute)
+        value = @instance.send(attribute)
+        @attributes << [attribute, value] if value
+      end
+
+      def visit_attribute_time(attribute)
+        visit_attribute(attribute)
+      end
+
+      def visit_attribute_enum(attribute, type)
+        visit_attribute(attribute)
+      end
+
+      def visit_process_reference(attribute)
+        process = @instance.send(attribute)
+        @attributes << [attribute, process.uuid] if process
+      end
+
+      def visit_task_reference(attribute)
+        task = @instance.send(attribute)
+        @attributes << [attribute, task.uuid] if task
+      end
+
+      def visit_type(attribute)
+        type = @instance.send(attribute)
+        @attributes << [attribute, type.name] if type
+      end
+
+      def visit_args(attribute)
+        values = @instance.send(attribute)
+        yaml = Taskinator::Persistence.serialize(values)
+
+        # greater than 2 MB?
+        if (yaml.bytesize / (1024.0**2)) > 2
+          Taskinator.logger.warn("Large argument data detected for '#{self.to_s}'. Consider using intrinsic types instead, or try to reduce the amount of data provided.")
+        end
+
+        @attributes << [attribute, yaml]
+      end
+
+      def task_count
+        @task_count
+      end
+
+      def incr_task_count
+        @task_count += 1
+      end
+    end
+
     class RedisDeserializationVisitor < Taskinator::Visitor::Base
 
       #

data/lib/taskinator/process.rb

@@ -41,6 +41,7 @@ module Taskinator
       @queue = options.delete(:queue)
       @created_at = Time.now.utc
       @updated_at = created_at
+      @current_state = :initial
     end
 
     def parent=(value)

data/lib/taskinator/task.rb

@@ -39,6 +39,7 @@ module Taskinator
       @queue = options.delete(:queue)
       @created_at = Time.now.utc
       @updated_at = created_at
+      @current_state = :initial
     end
 
     def accept(visitor)

data/lib/taskinator/version.rb

@@ -1,3 +1,3 @@
 module Taskinator
-  VERSION = "0.3.9"
+  VERSION = "0.3.10"
 end

data/lib/taskinator/workflow.rb

@@ -2,11 +2,11 @@ module Taskinator
   module Workflow
 
     def current_state
-      # NB: don't memoize this value (i.e. re-read it each time)
-      @current_state = load_workflow_state
+      @current_state ||= load_workflow_state
     end
 
     def current_state=(new_state)
+      return if new_state == @current_state
       @current_state = persist_workflow_state(new_state)
     end
 

data/spec/taskinator/persistence_spec.rb

@@ -210,6 +210,10 @@ describe Taskinator::Persistence, :redis => true do
       pending __FILE__
     end
 
+    describe "#to_xml" do
+      pending __FILE__
+    end
+
     describe "#key" do
       it {
         expect(subject.key).to match(/#{subject.uuid}/)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: taskinator
 version: !ruby/object:Gem::Version
-  version: 0.3.9
+  version: 0.3.10
 platform: ruby
 authors:
 - Chris Stefano
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-12 00:00:00.000000000 Z
+date: 2016-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis