Skip to content

Remove usage of Dask's get_worker#1365

Merged
rapids-bot[bot] merged 6 commits intorapidsai:branch-23.04from
pentschev:update-get_worker-usage
Mar 22, 2023
Merged

Remove usage of Dask's get_worker#1365
rapids-bot[bot] merged 6 commits intorapidsai:branch-23.04from
pentschev:update-get_worker-usage

Conversation

@pentschev
Copy link
Copy Markdown
Member

In dask/distributed#7580 get_worker was modified to return the worker of a task, thus it cannot be used by client.run, and we must now use dask_worker as the first argument to client.run to obtain the worker.

In dask/distributed#7580 get_worker was modified to return the worker of
a task, thus it cannot be used by client.run, and we must now use
dask_worker as the first argument to client.run to obtain the worker.
@cjnolet cjnolet added bug Something isn't working non-breaking Non-breaking change labels Mar 22, 2023
@pentschev pentschev requested a review from a team as a code owner March 22, 2023 17:47
@github-actions github-actions bot added the ci label Mar 22, 2023
Copy link
Copy Markdown
Member

@ajschmidt8 ajschmidt8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving ops-codeowner file changes

@cjnolet
Copy link
Copy Markdown
Member

cjnolet commented Mar 22, 2023

/merge

@rapids-bot rapids-bot bot merged commit a7e619c into rapidsai:branch-23.04 Mar 22, 2023
@cjnolet cjnolet mentioned this pull request Mar 23, 2023
@pentschev pentschev deleted the update-get_worker-usage branch May 13, 2024 17:43
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
In dask/distributed#7580 get_worker was modified to return the worker of a task, thus it cannot be used by client.run, and we must now use dask_worker as the first argument to client.run to obtain the worker.

Authors:
  - Peter Andreas Entschev (https://github.com/pentschev)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)
  - AJ Schmidt (https://github.com/ajschmidt8)

URL: rapidsai#1365
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci non-breaking Non-breaking change python

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants