default filename in cmd_args property in Python client (#374)

Netflix · Aug 24, 2016 · 3965658 · 3965658
1 parent 4edfd6b
commit 3965658
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/genie-client/src/main/python/pygenie/jobs/hive.py b/genie-client/src/main/python/pygenie/jobs/hive.py
@@ -56,11 +56,11 @@ def cmd_args(self):
         if self._command_arguments is not None:
             return self._command_arguments
 
+        filename = HiveJob.DEFAULT_SCRIPT_NAME
         if is_file(self._script):
             filename = os.path.basename(self._script)
             self._add_dependency(self._script)
         elif self._script is not None:
-            filename = HiveJob.DEFAULT_SCRIPT_NAME
             self._add_dependency({'name': filename, 'data': self._script})
 
         params_str = ' '.join([

diff --git a/genie-client/src/main/python/pygenie/jobs/pig.py b/genie-client/src/main/python/pygenie/jobs/pig.py
@@ -58,11 +58,11 @@ def cmd_args(self):
         if self._command_arguments is not None:
             return self._command_arguments
 
+        filename = PigJob.DEFAULT_SCRIPT_NAME
         if is_file(self._script):
             filename = os.path.basename(self._script)
             self._add_dependency(self._script)
         elif self._script is not None:
-            filename = PigJob.DEFAULT_SCRIPT_NAME
             self._add_dependency({'name': filename, 'data': self._script})
 
         param_files_str = ' '.join([

diff --git a/genie-client/src/main/python/pygenie/jobs/presto.py b/genie-client/src/main/python/pygenie/jobs/presto.py
@@ -64,11 +64,11 @@ def cmd_args(self):
         if self._command_arguments is not None:
             return self._command_arguments
 
+        filename = PrestoJob.DEFAULT_SCRIPT_NAME
         if is_file(self._script):
             filename = os.path.basename(self._script)
             self._add_dependency(self._script)
         elif self._script is not None:
-            filename = PrestoJob.DEFAULT_SCRIPT_NAME
             if not self._script.strip().endswith(';'):
                 self._script = '{};'.format(self._script)
             self._add_dependency({'name': filename, 'data': self._script})

diff --git a/genie-client/src/main/python/setup.py b/genie-client/src/main/python/setup.py
@@ -26,7 +26,7 @@
 
 setup(
     name='nflx-genie-client',
-    version='3.0.33',
+    version='3.0.34',
     author='Netflix Inc.',
     author_email='genieoss@googlegroups.com',
     keywords='genie hadoop cloud netflix client bigdata presto',