claudevdm commented on code in PR #36709: URL: https://github.com/apache/beam/pull/36709#discussion_r2487889109
########## sdks/python/apache_beam/internal/cloudpickle/cloudpickle.py: ########## @@ -169,6 +169,17 @@ class CloudPickleConfig: DEFAULT_CONFIG = CloudPickleConfig() + +# Minimal helper to return the provided object during unpickling. +def _return_obj(obj): + return obj + + +# Optional import for Python 3.12 TypeAliasType +try: # pragma: no cover - dependent on Python version Review Comment: Can you rather add the custom reducer to the dispatch table like https://github.com/apache/beam/blob/89947fded65065c8770a35096f2074c729b239dc/sdks/python/apache_beam/internal/cloudpickle_pickler.py#L171 I dont think we need to change the underlying cloudpickle library for this. -- 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]
