postjob 0.5.3 → 0.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ddb381c651727adfa2e654b8e9902510ede3dad3
-  data.tar.gz: e84c5cace159a3b39ce3cb34fc479d882eb4147a
+  metadata.gz: f08479560788fb4e593c4c0ff8d1d70c54ff7f17
+  data.tar.gz: 393afe819d58f03bcff1314f819786be4c31cbc7
 SHA512:
-  metadata.gz: 7d05ff63ead66235f51259cf86ff4565fc2aa8cee2583dcf83bcfc3767b9685b6c91c100b9d86e230f888eb9ca28e4d1a5aae36a4d9b18bf25d23de9314ff703
-  data.tar.gz: de96874a8e293a2357bda35885f55aa14b152e9f849d796de98596edeec8a6481bea7c354a293dcaa02bd570ae94a58c96d0134a897a7e75273a04c5f26be48d
+  metadata.gz: 3b2cb1875294c491d6e38d8cfb196809bb221f1f92359c063f010620e2a044d602b968295617914b23aecbb314ebfaa53743d8a4e15b2aaadc5516b08ebfc2ef
+  data.tar.gz: 319849a057e3d791faabc7bf3f03e96a74a5b4f85858efee8af4efee119a2ae83035d73986e047365180f501cfca6a41f70a9731f1499fd39ab312df19a160a8

data/lib/postjob/queue/notifications.rb

@@ -45,6 +45,6 @@ module Postjob::Queue::Notifications
   # next runnable or timeoutable job comes up.
   def time_to_next_job(worker_session_id)
     expect! worker_session_id => /[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/i
-    Simple::SQL.ask "SELECT * FROM #{SCHEMA_NAME}.time_to_next_job($1)", worker_session_id
+    Simple::SQL.ask "SELECT * FROM #{SCHEMA_NAME}.time_to_next_job($1::uuid)", worker_session_id
   end
 end

data/lib/postjob/queue.rb

@@ -1,6 +1,7 @@
 # rubocop:disable Layout/AlignParameters
 # rubocop:disable Metrics/MethodLength
 # rubocop:disable Metrics/ParameterLists
+# rubocop:disable Metrics/LineLength
 
 require "securerandom"
 
@@ -50,7 +51,7 @@ module Postjob::Queue
     # a) a limitation in Simple::SQL which would not be able to unpack a
     #    "SELECT function()" usefully when the return value is a record;
     # b) and/or my inability to write better SQL functions;
-    SQL.ask "SELECT * FROM #{SCHEMA_NAME}.enqueue($1, $2, $3, $4, $5, $6, $7, $8, $9, $10)",
+    SQL.ask "SELECT * FROM #{SCHEMA_NAME}.enqueue($1::uuid, $2, $3, $4, $5, $6, $7, $8, $9, $10)",
                   worker_session_id,
                   options[:queue],
                   workflow,
@@ -66,17 +67,17 @@ module Postjob::Queue
 
   def set_job_result(worker_session_id, job, value, version:)
     value = Encoder.encode([value]) unless value.nil?
-    SQL.ask "SELECT #{SCHEMA_NAME}.set_job_result($1, $2, $3, $4)", worker_session_id, job.id, value, version
+    SQL.ask "SELECT #{SCHEMA_NAME}.set_job_result($1::uuid, $2, $3, $4)", worker_session_id, job.id, value, version
   end
 
   def set_job_pending(worker_session_id, job, version:)
-    SQL.ask "SELECT #{SCHEMA_NAME}.set_job_pending($1, $2, $3)", worker_session_id, job.id, version
+    SQL.ask "SELECT #{SCHEMA_NAME}.set_job_pending($1::uuid, $2, $3)", worker_session_id, job.id, version
   end
 
   def set_job_error(worker_session_id, job, error, error_message, error_backtrace = nil, status:, version:)
     expect! status => [ :failed, :err, :timeout ]
 
-    SQL.ask "SELECT #{SCHEMA_NAME}.set_job_error($1, $2, $3, $4, $5, $6, $7, $8)",
+    SQL.ask "SELECT #{SCHEMA_NAME}.set_job_error($1::uuid, $2, $3, $4, $5, $6, $7, $8)",
       worker_session_id, job.id, error, error_message, Encoder.encode(error_backtrace), status, version, Postjob.fast_mode
   end
 
@@ -107,18 +108,18 @@ module Postjob::Queue
     # a) a limitation in Simple::SQL which would not be able to unpack a
     #    "SELECT function()" usefully when the return value is a record;
     # b) and/or my inability to write better SQL functions;
-    return SQL.ask "SELECT * FROM #{SCHEMA_NAME}.find_or_create_childjob($1, $2, $3, $4, $5, $6, $7, $8, $9, $10)",
-                  worker_session_id,
-                  queue,
-                  workflow,
-                  workflow_method,
-                  nil,                      # version
-                  Encoder.encode(args),
-                  parent.id,
-                  nil,                      # tags will be read from parent
-                  max_attempts,
-                  timeout,
-                  into: Job
+    SQL.ask "SELECT * FROM #{SCHEMA_NAME}.find_or_create_childjob($1::uuid, $2, $3, $4, $5, $6, $7, $8, $9, $10)",
+           worker_session_id,
+           queue,
+           workflow,
+           workflow_method,
+           nil,                      # version
+           Encoder.encode(args),
+           parent.id,
+           nil,                      # tags will be read from parent
+           max_attempts,
+           timeout,
+           into: Job
   end
 
   private
@@ -139,7 +140,7 @@ module Postjob::Queue
 
   def checkout(worker_session_id)
     expect! worker_session_id => UUID_REGEXP
-    SQL.ask "SELECT * FROM #{SCHEMA_NAME}.checkout($1, $2)", worker_session_id, Postjob.fast_mode, into: Job
+    SQL.ask "SELECT * FROM #{SCHEMA_NAME}.checkout($1::uuid, $2::boolean)", worker_session_id, Postjob.fast_mode, into: Job
   end
 
   def find_or_create_token(job)
@@ -164,7 +165,7 @@ module Postjob::Queue
   def start_worker_session(workflows_with_versions, host_id:)
     expect! host_id => UUID_REGEXP
 
-    Simple::SQL.ask "SELECT * FROM postjob.worker_session_start($1, $2)", host_id, workflows_with_versions, into: ::Postjob::WorkerSession
+    Simple::SQL.ask "SELECT * FROM postjob.worker_session_start($1::uuid, $2)", host_id, workflows_with_versions, into: ::Postjob::WorkerSession
   end
 
   # sends in a heartbeat

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: postjob
 version: !ruby/object:Gem::Version
-  version: 0.5.3
+  version: 0.5.4
 platform: ruby
 authors:
 - radiospiel
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-27 00:00:00.000000000 Z
+date: 2018-07-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec