From: Peter Amstutz <peter.amstutz@curoverse.com>
Date: Mon, 7 Jul 2014 20:24:04 +0000 (-0400)
Subject: Google API client jsonifies the body, so we don't have to.
X-Git-Tag: 1.1.0~2467^2
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/3d2a90bcc1c96a8f3b920038aba264069307a609

Google API client jsonifies the body, so we don't have to.
---

diff --git a/sdk/cli/bin/arv-run-pipeline-instance b/sdk/cli/bin/arv-run-pipeline-instance
index 30db8d7c5f..fecee0b57b 100755
--- a/sdk/cli/bin/arv-run-pipeline-instance
+++ b/sdk/cli/bin/arv-run-pipeline-instance
@@ -325,12 +325,7 @@ class JobCache
   def self.create(pipeline, component, job, create_params)
     @cache ||= {}
 
-    jsonified_create_params = {}
-    create_params.each do |k, v|
-      jsonified_create_params[k] = v.to_json unless v.nil?
-    end
-
-    body = { :job => job }.merge(jsonified_create_params)
+    body = { :job => job }.merge(create_params.select { |k,v| not v.nil? })
 
     result = $client.execute(:api_method => $arvados.jobs.create,
                              :body => body,