robertwb commented on pull request #12560: URL: https://github.com/apache/beam/pull/12560#issuecomment-685161062
The issue this is solving is when a `DeferredBase` gets packaged up as part of the environment (e.g. in globals, or as an attribute on a PTransform or DoFn, etc.) Given how pickling tries to find the transitive closure of reachable things, this can happen by accident (and can require contortions to avoid), so shouldn't be an error (as long as you don't try to use it). We do the same thing for PCollection: https://github.com/apache/beam/blob/master/sdks/python/apache_beam/pvalue.py#L170. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
