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, Dec 27, 4:05 AM
Unknown Object (File)
Sat, Dec 14, 10:56 PM
Unknown Object (File)
Mon, Dec 9, 12:11 AM
Unknown Object (File)
Sat, Dec 7, 11:35 AM
Unknown Object (File)
Sun, Dec 1, 1:41 AM
Unknown Object (File)
Nov 26 2024, 4:56 PM
Unknown Object (File)
Nov 20 2024, 7:22 PM
Unknown Object (File)
Nov 18 2024, 5:57 PM
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.