10000 [core] Static Priority Scheduling (3/N) by dayshah · Pull Request #52506 · ray-project/ray · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[core] Static Priority Scheduling (3/N) #52506

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 11 commits into
base: master
Choose a base branch
from
Draft

[core] Static Priority Scheduling (3/N) #52506

wants to merge 11 commits into from

Conversation

dayshah
Copy link
Contributor
@dayshah dayshah commented Apr 21, 2025

Why are these changes needed?

Related issue number

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

dayshah added 6 commits April 19, 2025 21:28
@dayshah
Get priority from python to cpp
6c40165 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah
normal_task_submitter changes
04d113d 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah
fix
0606466 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah
fix for cpp test
0ad5cd8 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah
cluster task manager changes
8f6a1ec 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah
build without local task manager changes
9ee2515 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah dayshah changed the title [core] Priority scheduling (3/N) [core] Static Priority Scheduling (3/N) Apr 21, 2025
@dayshah
fix
cfc99b2 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah dayshah force-pushed the priority-scheduling-2.5 branch from a41a2d2 to c070987 Compare April 21, 2025 20:29
@dayshah
order by priority more and fix build dep
69cea6f 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah dayshah force-pushed the priority-scheduling-2.5 branch from c070987 to 5995c5e Compare April 21, 2025 21:02
@dayshah
fix build
0405b1e 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah dayshah force-pushed the priority-scheduling-2.5 branch from 5995c5e to 196777a Compare April 21, 2025 22:05
dayshah added 2 commits April 22, 2025 13:33
@dayshah
fix infeasible logic
e92cf24 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah
local task manager changes
b42a311 
Signed-off-by: dayshah <dhyey2019@gmail.com>
@dayshah dayshah force-pushed the priority-scheduling-2.5 branch from 196777a to b42a311 Compare April 22, 2025 20:34
@github-actions GitHub Actions
Copy link
github-actions bot commented Jun 9, 2025

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

@github-actions github-actions bot added the stale The issue is stale. It will be closed within 7 days unless there are further conversation label Jun 9, 2025
@dayshah dayshah removed the stale The issue is stale. It will be closed within 7 days unless there are further conversation label Jun 9, 2025
@github-actions GitHub Actions
Copy link

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

@github-actions github-actions bot added the stale The issue is stale. It will be closed within 7 days unless there are further conversation label Jun 24, 2025
@dayshah dayshah removed the stale The issue is stale. It will be closed within 7 days unless there are further conversation label Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pcmoritz pcmoritz Awaiting requested review from pcmoritz pcmoritz will be requested when the pull request is marked ready for review pcmoritz is a code owner

@raulchen raulchen Awaiting requested review from raulchen raulchen will be requested when the pull request is marked ready for review raulchen is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dayshah @hainesmichaelc
0