tvalentyn commented on code in PR #26099: URL: https://github.com/apache/beam/pull/26099#discussion_r1157348236
########## sdks/python/apache_beam/coders/coders.py: ########## @@ -78,14 +78,16 @@ # pylint: disable=wrong-import-order, wrong-import-position # Avoid dependencies on the full SDK. -try: - # Import dill from the pickler module to make sure our monkey-patching of dill - # occurs. - from apache_beam.internal.dill_pickler import dill -except ImportError: - # We fall back to using the stock dill library in tests that don't use the - # full Python SDK. - import dill + Review Comment: Let's try to keep this file as is and include dill as a test dependency. -- 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]
