Make run id optional when using http client (#112)

wslulciuc · web-flow · commit c60845f3645f · 2020-10-13T12:17:10.000-07:00
Signed-off-by: wslulciuc &lt;willy@datakin.com&gt;
diff --git a/marquez_client/client.py b/marquez_client/client.py
@@ -237,7 +237,7 @@ def list_jobs(self, namespace_name, limit=None, offset=None):
             }
         )
 
-    def create_job_run(self, namespace_name, job_name, run_id,
+    def create_job_run(self, namespace_name, job_name, run_id=None,
                        nominal_start_time=None,
                        nominal_end_time=None, run_args=None,
                        mark_as_running=False):
@@ -246,7 +246,8 @@ def create_job_run(self, namespace_name, job_name, run_id,
 
         payload = {}
 
-        payload['id'] = run_id
+        if run_id:
+            payload['id'] = run_id
 
         if nominal_start_time:
             payload['nominalStartTime'] = nominal_start_time

Original file line number	Diff line number	Diff line change
`@@ -237,7 +237,7 @@ def list_jobs(self, namespace_name, limit=None, offset=None):`
`237`	`237`	`}`
`238`	`238`	`)`
`239`	`239`
`240`		`- def create_job_run(self, namespace_name, job_name, run_id,`
	`240`	`+ def create_job_run(self, namespace_name, job_name, run_id=None,`
`241`	`241`	`nominal_start_time=None,`
`242`	`242`	`nominal_end_time=None, run_args=None,`
`243`	`243`	`mark_as_running=False):`
`@@ -246,7 +246,8 @@ def create_job_run(self, namespace_name, job_name, run_id,`
`246`	`246`
`247`	`247`	`payload = {}`
`248`	`248`
`249`		`- payload['id'] = run_id`
	`249`	`+ if run_id:`
	`250`	`+ payload['id'] = run_id`
`250`	`251`
`251`	`252`	`if nominal_start_time:`
`252`	`253`	`payload['nominalStartTime'] = nominal_start_time`