[Compass-ci] Re: [PATCH v2 compass-ci] sched: refactor sched class for the find job boot function

On Thu, Nov 05, 2020 at 07:40:46PM +0800, Xiao Shenwei wrote:

On Thu, Nov 05, 2020 at 06:01:41PM +0800, Cao Xueliang wrote:

...

According to scheduler.cr API to refactor sched class.

Extract find_job_boot function from sched.cr to find_job_boot.cr Extract find_next_job_boot function from sched.cr to find_next_job_boot.cr

Signed-off-by: Cao Xueliang caoxl78320@163.com

src/lib/sched.cr | 237 +--------------------------- src/scheduler/find_job_boot.cr | 221 ++++++++++++++++++++++++++ src/scheduler/find_next_job_boot.cr | 14 ++

find_job_boot.cr and find_next_job_boot.cr can merge one file find_job.cr?

these 2 file similar functions

We according to scheduler.cr API to refactor sched class.

One API use one file.

Thanks, Xueliang

Thanks, Shenwei