[BEAM-7253] test_with_jar_packages_invalid_file_name test fails on wi…

…ndows Using ',' for separator
grtheod · May 13, 2019 · d618dc1 · d618dc1
1 parent 54c75de
commit d618dc1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/sdks/python/apache_beam/runners/portability/stager.py b/sdks/python/apache_beam/runners/portability/stager.py
@@ -203,7 +203,7 @@ def stage_job_resources(self,
     if jar_packages is not None:
       resources.extend(
           self._stage_jar_packages(
-              jar_packages.split(':'), staging_location,
+              jar_packages.split(','), staging_location,
               temp_dir=temp_dir))
 
     # Pickle the main session if requested.

diff --git a/sdks/python/apache_beam/runners/portability/stager_test.py b/sdks/python/apache_beam/runners/portability/stager_test.py
@@ -587,7 +587,7 @@ def test_with_jar_packages(self):
     options = PipelineOptions()
     self.update_options(options)
     options.view_as(DebugOptions).experiments = [
-        'jar_packages=%s:%s:%s:%s' % (
+        'jar_packages=%s,%s,%s,%s' % (
             os.path.join(source_dir, 'abc.jar'),
             os.path.join(source_dir, 'xyz.jar'),
             os.path.join(source_dir, 'ijk.jar'),