You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by al...@apache.org on 2020/04/03 22:05:16 UTC

[beam] branch master updated: Change delimeter to a dash as it is a reserved symbol in Windows

This is an automated email from the ASF dual-hosted git repository.

altay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new c81c468  Change delimeter to a dash as it is a reserved symbol in Windows
     new a723148  Merge pull request #11306 from rohdesamuel/fix-delim
c81c468 is described below

commit c81c4683b95ac7af57811ec0e7837874b512abc1
Author: Sam Rohde <ro...@gmail.com>
AuthorDate: Fri Apr 3 10:37:38 2020 -0700

    Change delimeter to a dash as it is a reserved symbol in Windows
    
    Change-Id: I96853c57f421f60ddbee24c1b4f94ec5b80d948c
---
 sdks/python/apache_beam/runners/interactive/pipeline_instrument.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sdks/python/apache_beam/runners/interactive/pipeline_instrument.py b/sdks/python/apache_beam/runners/interactive/pipeline_instrument.py
index 9b97c50..c62110a 100644
--- a/sdks/python/apache_beam/runners/interactive/pipeline_instrument.py
+++ b/sdks/python/apache_beam/runners/interactive/pipeline_instrument.py
@@ -73,11 +73,11 @@ class CacheKey:
 
   @staticmethod
   def from_str(r):
-    split = r.split('|')
+    split = r.split('-')
     return CacheKey(split[0], split[1], split[2], split[3])
 
   def __repr__(self):
-    return '|'.join(
+    return '-'.join(
         [self.var, self.version, self.producer_version, self.pipeline_id])