JobWrapper 'DelayIfStillRunning' don't work using 'WithChain'

[flpsaguiar]

Hi Robfig,

i'm trying to use the JobWrapper "DelayIfStillRunning", together with "WithChain", but it is not working (or it can't be used the way I'm trying to). I tested the JobWrapper "SkipIfStillRunning" and it works.

Here's an example of how I'm trying to use it.

c := cron.New(cron.WithChain(
	cron.DelayIfStillRunning(cron.DiscardLogger),
))
c.Start()

c.AddJob("17 * * * *", firstJob)
c.AddJob("18 * * * *", secondJob)

"firstJob" has an infinite loop in the Run function and even then the second Job is started.

[KevinWu0904]

@flpsaguiar I thought you misunderstanding "DelayIfStillRunning". It was not used for different jobs, but for one job's adjacent triggers.

For your example：

c := cron.New(cron.WithChain(
	cron.DelayIfStillRunning(cron.DiscardLogger),
))
c.Start()

c.AddJob("17 * * * *", firstJob)

"firstJob" has an infinite loop in the Run function. OK, then the job will tigger every minute.

If you don't set DelayIfStillRunning, then it will trigger two concurrent Run function to execute each infinite loop. If you set DelayIfStillRunning, then the new function won't start until the first one done.