Skip to content
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

Revert "drop deprecated tornado.netutil.ExecutorResolver" #6117

Conversation

graingert
Copy link
Member

@graingert graingert commented Apr 13, 2022

Fixes #6087

Reverts #6031

@mrocklin
Copy link
Member

Please no one merge this. I'm going to try to work around this before next release. If I can't then I'll merge this.

@github-actions
Copy link
Contributor

Unit Test Results

       16 files  ±0         16 suites  ±0   7h 29m 40s ⏱️ - 4m 35s
  2 737 tests ±0    2 654 ✔️ ±0       80 💤  - 1  3 +1 
21 781 runs  ±0  20 705 ✔️ +1  1 073 💤  - 2  3 +1 

For more details on these failures, see this check.

Results for commit 2f2beb1. ± Comparison against base commit 7d55039.

@mrocklin
Copy link
Member

Closing this for now. We'll see how the alternative fares.

@mrocklin mrocklin closed this Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error closing a local cluster when client still running
2 participants