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.

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

Refactor Queue to a single set of slots #427

Closed
Tracked by #424
rtholmes opened this issue Oct 12, 2022 · 1 comment
Closed
Tracked by #424

Refactor Queue to a single set of slots #427

rtholmes opened this issue Oct 12, 2022 · 1 comment
Assignees
@rtholmes
Milestone
v2.5

Comments

@rtholmes
Copy link
Contributor

As a consequence of the Queue refactor from #390 and #392, Queue clearly has too much information about executing and there should be a common job pool instead of just a single set of execution slots that are pulled from the different priority queues.

This will simplify Queue, simplify AutoTest, and make it possible to have a more dynamic number of slots in the future.
@rtholmes rtholmes added this to the v2.5 milestone Oct 12, 2022
@rtholmes rtholmes self-assigned this Oct 12, 2022
@rtholmes rtholmes mentioned this issue Oct 12, 2022
Merged
5 tasks
rtholmes added a commit that referenced this issue Oct 15, 2022
@rtholmes
see #427 queue refactor to single job pool
b9ffee8
rtholmes added a commit that referenced this issue Oct 15, 2022
@rtholmes
see #427 make numJobs configurable; simplify queue handling code. ren…
d12eccd 
…ame slots -> jobs, since that's what they are.
@rtholmes
Copy link
Contributor Author

Fixed in #424

@rtholmes rtholmes mentioned this issue Nov 3, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rtholmes rtholmes

Labels
None yet
Projects
None yet
Milestone
v2.5
Development

No branches or pull requests
1 participant
@rtholmes