-
Notifications
You must be signed in to change notification settings - Fork 111
Provide client in activity context #740
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
962135c
to
3164f69
Compare
This needs a bit more work -- as things stand it's attempting to pickle the client in the context in tests of process-based executor pools (which fails when it tries to pickle the |
abf45cb
to
4577018
Compare
4577018
to
a8c177e
Compare
I'd be ok for now raising an exception for if the |
bd3b25e
to
2f04c55
Compare
2f04c55
to
3c2c1bc
Compare
Rebased and ready for review again. Makes the worker's connected As discussed above, the client is only available in |
3c2c1bc
to
08e0223
Compare
fc616a6
to
7ef37f6
Compare
Fixes #721