You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/01/21 23:02:23 UTC

[GitHub] [beam] tvalentyn commented on a change in pull request #16589: [BEAM-13595] Set pickle library to cloud pickle

tvalentyn commented on a change in pull request #16589:
URL: https://github.com/apache/beam/pull/16589#discussion_r790048677



##########
File path: sdks/python/apache_beam/runners/worker/sdk_worker_main.py
##########
@@ -252,7 +252,8 @@ def _load_main_session(semi_persistent_directory):
     else:
       _LOGGER.warning(
           'No session file found: %s. Functions defined in __main__ '
-          '(interactive session) may fail.',
+          '(interactive session) may fail. Ignore this warning'

Review comment:
       We should be able to skip the `_load_main_session` call or not print the warning, so that the users don't have to worry about understanding what pickle_library is when we switch the default to cloudpickle.




-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org