humanmade / Cavalcade

A better wp-cron. Horizontally scalable, works perfectly with multisite.
https://engineering.hmn.md/projects/cavalcade/
Other
515 stars 46 forks source link

Runner\get_next_job() should explicitely state an order #38

Closed dd32 closed 7 years ago

dd32 commented 7 years ago

Currently get_next_job() does two things wrong IMHO:

The ORDER BY shouldn't be an issue, except when backlogged by something such as #37

rmccue commented 7 years ago

I feel like I chose not to include the LIMIT for some reason, but cannot remember why. In any case, 👍 on changing it.

Ditto 👍 on adding ORDER BY