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 2018/07/13 22:40:45 UTC

[beam] branch master updated: [BEAM-4752] Add dill compatibility for older versions of dill (#5949)

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 2ca320c  [BEAM-4752] Add dill compatibility for older versions of dill (#5949)
2ca320c is described below

commit 2ca320cc4265d2bdc8b678579b01597cea3cd837
Author: Ahmet Altay <aa...@gmail.com>
AuthorDate: Fri Jul 13 15:40:39 2018 -0700

    [BEAM-4752] Add dill compatibility for older versions of dill (#5949)
    
    * Add dill compatibility for older versions of dill
---
 sdks/python/apache_beam/internal/pickler.py | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/sdks/python/apache_beam/internal/pickler.py b/sdks/python/apache_beam/internal/pickler.py
index a4d20b9..211430b 100644
--- a/sdks/python/apache_beam/internal/pickler.py
+++ b/sdks/python/apache_beam/internal/pickler.py
@@ -44,6 +44,12 @@ import dill
 # TODO: Remove this once Beam depends on dill >= 0.2.8
 if not getattr(dill, 'dill', None):
   dill.dill = dill._dill
+  sys.modules['dill.dill'] = dill._dill
+
+# TODO: Remove once Dataflow has containers with a preinstalled dill >= 0.2.8
+if not getattr(dill, '_dill', None):
+  dill._dill = dill.dill
+  sys.modules['dill._dill'] = dill.dill
 
 
 def _is_nested_class(cls):