Skip to content

Conversation

@VictorVerhaert
Copy link
Contributor

No description provided.

:param parallel_jobs:
Maximum number of jobs to allow in parallel on a backend.
:param queueing_limit:
Maximum number of jobs to allow in queue on a backend. Setting this too high may lead to a backend rejecting jobs.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Maximum number of jobs to allow in queue on a backend. Setting this too high may lead to a backend rejecting jobs.
Maximum number of queued jobs (on the backend) the job manager should stay below before starting more jobs. Setting this too high may lead to the backend rejecting jobs.

@VictorVerhaert
Copy link
Contributor Author

General note on this pr. As the queued limit is not a limit backends should enforce on top of the job concurrency and only a temporary fix, @HansVRP made a good point of not exposing this as a public param so we can revert this safeguard later on without risking backwards compatability.

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.

3 participants