Skip to content
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.

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

Add balanced mode to dispatch query #1103

Conversation

DiegoTavares
Copy link
Collaborator

Link the Issue(s) this Pull Request is related to.
Large studios with multiple shows and nodes can experience resource starving in smaller shows due to the priority model

Summarize your change.

Adds a scheduler_mode option to choose between BALANCED, PRIORITY_ONLY and FIFO.

  • PRIORITY_ONLY was the default mode
  • FIFO replaces the property fifo_scheduling_enabled and uses its logic
  • BALANCED described bellow

The new mode (BALANCED) uses a formula that makes sure smaller jobs don't get starved by large jobs and also takes into account the time a job has been waiting in the queue. This mode also disregards the layer_limit feature for performance implications. The formula takes into account the number of required cores and the time the job has been waiting on queue.
Formula:

  rank = priority + (100 * (1 - (job.cores/job.int_min_cores))) + (age in days)

This heuristic favours scenarios with a large amount of jobs and hosts.

Dispatch balanced mode switch. When turned on, the dispatch query will
use a formula that makes sure smaller jobs don't get starved by large jobs.
This mode also disregards the layer_limit feature for performance implications.
the formula takes into account the number of required cores and the time the
job has been waiting on queue.  Formula:
  rank = priority + (100 * (1 - (job.cores/job.int_min_cores))) + (age in days)
This heuristic favors scenarios with a large amount of jobs and hosts and
when turned off, only job priorities are taken into account, making the queue more predictable.
@DiegoTavares DiegoTavares changed the title Draft: Add balanced mode to dispatch query Add balanced mode to dispatch query Mar 28, 2022
@DiegoTavares
Copy link
Collaborator Author

@bcipriano
We're finally merging our version of the dispatch query in an option selectable way.

@DiegoTavares
Copy link
Collaborator Author

@bcipriano would you review this

@DiegoTavares DiegoTavares merged commit 351c9ae into AcademySoftwareFoundation:master Apr 11, 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.

None yet

2 participants