Skip to content

Add worker memory and processing RPC handlers#5642

Open
jacobtomlinson wants to merge 2 commits intodask:mainfrom
jacobtomlinson:worker-memory-processing-rpc
Open

Add worker memory and processing RPC handlers#5642
jacobtomlinson wants to merge 2 commits intodask:mainfrom
jacobtomlinson:worker-memory-processing-rpc

Conversation

@jacobtomlinson
Copy link
Copy Markdown
Member

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

Adds a couple more handlers to support the dask-ctl dashboard. Still needs tests.

@jacobtomlinson jacobtomlinson marked this pull request as ready for review January 13, 2022 10:26
@jacobtomlinson
Copy link
Copy Markdown
Member Author

Added tests now so this is ready for review.

@fjetter
Copy link
Copy Markdown
Member

fjetter commented Jan 24, 2022

Would it make sense to define these as a WorkerPlugin as part of dask-ctl since it is the only user?

@jacobtomlinson
Copy link
Copy Markdown
Member Author

You mean a SchedulerPlugin? I'd rather avoid that as it would require dask-ctl to be installed on the scheduler system.

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