onyx icon indicating copy to clipboard operation
onyx copied to clipboard

Co-located task scheduler does not respect capacity contrains

Open RolT opened this issue 6 years ago • 0 comments

Hello,

The :onyx.task-scheduler/colocated scheduler does not respect tasks' :onyx/max-peers constrain. I do not know if there is a reason for that. The code seems to explicitly prevent it: https://github.com/onyx-platform/onyx/blob/0.14.x/src/onyx/scheduling/colocated_task_scheduler.clj#L114-L116

Is it a feature ? If so could we make it an option (I'd be happy to provide a PR) ? I need a version of this scheduler that respects tasks capacity constrains. Is there something wrong in just returning true here instead ? I did a few tests with this modification, simulating multiple peers with docker, and there was no errors.

Any thoughts ?

RolT avatar Oct 30 '18 15:10 RolT