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

Remove leaking reference to workers from gen_cluster #6337

Merged
merged 3 commits into from
May 13, 2022

Conversation

hendrikmakait
Copy link
Member

@hendrikmakait hendrikmakait commented May 13, 2022

Extracted from #6336 to merge independently and enable using this for working on garbage collection of Worker instances.

  • Tests added / passed
  • Passes pre-commit run --all-files

@github-actions
Copy link
Contributor

Unit Test Results

       15 files  +       3         15 suites  +3   6h 59m 41s ⏱️ + 1h 35m 21s
  2 774 tests ±       0    2 693 ✔️ +     10    79 💤  -   12  2 +2 
20 580 runs  +3 962  19 671 ✔️ +3 847  907 💤 +113  2 +2 

For more details on these failures, see this check.

Results for commit 0e4175e. ± Comparison against base commit 2b04d8b.

@mrocklin mrocklin merged commit 3bedd8e into dask:main May 13, 2022
@mrocklin
Copy link
Member

Thanks @hendrikmakait !

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.

2 participants