schedule nextrun for NOW()+interval if nextrun+interval is in the past

humanmade / Cavalcade-Runner

Daemon for Cavalcade, a scalable WordPress jobs system.

https://engineering.hmn.md/projects/cavalcade/

53 stars 26 forks source link

schedule nextrun for NOW()+interval if nextrun+interval is in the past #57

Closed sirilyan closed 3 years ago

edubacco commented 5 years ago

I think that this is a good solution :+1:

archon810 commented 4 years ago

The patch has been here for 1.5 years. Is anyone going to merge it?

roborourke commented 3 years ago

This has been fixed at the PHP level in #64 by enforcing the use of UTC for calculating the interval just as the db does. We do still recommend having the Cavalcade Runner's server time set to UTC but acknowledge that's not always possible.