Skip to content

Conversation

@fjetter
Copy link
Member

@fjetter fjetter commented May 6, 2022

I removed Status.ANY_RUNNING recently for a more explicit WORKER_ANY_RUNNING.
Using the WORKER_ANY_RUNNING in the server core doesn't make any sense so I think just logging this message is OK

@crusaderky
Copy link
Collaborator

crusaderky commented May 6, 2022

While closing_gracefully and paused don't make sense here, shouldn't we still check for running? closing and closed are Server statuses

@github-actions
Copy link
Contributor

github-actions bot commented May 6, 2022

Unit Test Results

       16 files  +       16         16 suites  +16   7h 30m 1s ⏱️ + 7h 30m 1s
  2 760 tests +  2 760    2 678 ✔️ +  2 678       79 💤 +     79  3 +3 
22 042 runs  +22 042  21 016 ✔️ +21 016  1 022 💤 +1 022  4 +4 

For more details on these failures, see this check.

Results for commit a493bf1. ± Comparison against base commit 9f7646a.

@crusaderky crusaderky merged commit 8411c2d into dask:main May 8, 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.

2 participants