robertwb commented on a change in pull request #15357:
URL: https://github.com/apache/beam/pull/15357#discussion_r695283184
##########
File path: sdks/python/apache_beam/coders/coders.py
##########
@@ -18,6 +18,20 @@
"""Collection of useful coders.
Only those coders listed in __all__ are part of the public API of this module.
+
+## On usage of `pickle`, `dill` and `pickler` in Beam
+
+In Beam, we generally we use `pickle` for pipeline elements and `dill` for
+more complex types, like user functions.
+
+`pickler` is Beam's own wrapping of dill + compression + error handling.
+It serves also as an API to mask the actual encoding layer (so we can
+change it from `dill` if necessary).
+
+We created `_MemoizingPickleCoder` to improve performance when serializing
+complex user types for the execution of SDF. Specifically to address
+BEAM-12781, where `BoundedSource` instances are being encoded.
Review comment:
where many identical `BoundedSource` instances.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
