Skip to content

[core] [easy] readability improvements for IO Workers #52590

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

[core] [easy] readability improvements for IO Workers #52590

wants to merge 1 commit into from

Conversation

israbbani
Copy link
Contributor

Changes include:

  • Rename the max_io_workers to describe its purpose
  • Remove usage of RayConfig in WorkerPool and replace with the config being passed down
  • use move for pass-by-value parameters that are already copied and used once
  • include-what-you-use

@israbbani
Readability and style improvements
61444df 
- Rename max_io_workers to a more descriptive name
- Remove usage of RayConfig in WorkerPool and replace with the config
being passed down
- use move for pass-by-value parameters that are already copied and used
once
- include-what-you-use

Signed-off-by: irabbani <irabbani@anyscale.com>
@israbbani israbbani added the go add ONLY when ready to merge, run all tests label Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
go add ONLY when ready to merge, run all tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@israbbani