search

ru-fix / distributed-job-manager

DistributedJobManager will balance workload between servers for you.
https://ru-fix.github.io/distributed-job-manager
3 stars 1 forks source link

move all management of jobs executions to ScheduledJobManager #81

Closed KazankovMarch closed 3 years ago

KazankovMarch commented 4 years ago

I wanted to resolve issue #71, but management of jobs executions is separated between Worker and ScheduledJobManager

Worker is too complex class, so let's make that only ScheduledJobManager now responsible for management of jobs executions. Such a first step for refactoring Worker class

Note: I didn't change any DJM logic in this PR, I only moved some blocks of code from Worker to ScheduledJobManager; I suggest refactor ScheduledJobManager in next PR