Page MenuHomePhabricator

Make taskmaster consumption of failed tasks more FIFO-ey
ClosedPublic

Authored by epriestley on Dec 9 2013, 4:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Fri, Apr 19, 3:30 AM
Unknown Object (File)
Thu, Apr 18, 5:22 AM
Unknown Object (File)
Tue, Apr 16, 4:19 AM
Unknown Object (File)
Thu, Apr 11, 7:37 AM
Unknown Object (File)
Thu, Apr 11, 4:41 AM
Subscribers

Details

Summary

Ref T1049. See discussion in D7745. We have some specific interest in this for D7745, but generally we want to consume tasks with expired leases in roughly FIFO order, just like we consume new tasks in roughly FIFO order. Currently, when we select an expired task we order them by id, but this is the original insert order, not lease expiration order. Instead, order by leaseExpires.

This query is actually much better than the old one was, since the WHERE part is leaseExpries < VALUE.

Test Plan

Ran EXPLAIN on the query. Ran a taskmaster in debug mode and saw it lease new and expired tasks successfully.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Dec 9 2013, 4:16 AM
  • Add explicit test coverage. I was thinking this would be a pain to cover, but we actually have good infrastructure here already.