Homec4science

Fix daemon task queue to respect task priority

Authored by epriestley <git@epriestley.com> on Oct 31 2014, 17:27.

Description

Fix daemon task queue to respect task priority

Summary:
Fixes an issue with T5336 / D9871. We did 99% of the work here but didn't actually turn on the priority sorting. The unit test passed by default, which didn't catch this.

  • Fix the unit test (it failed).
  • Fix the query (test now passes).
  • Add a "Next in Queue" element to the UI to make this kind of thing easier to spot/understand.

Test Plan: Ran unit test. Viewed "Next in Queue". Queued some tasks, flushed the queue. Web UI tracked the state sensibly.

Reviewers: joshuaspence, btrahan

Reviewed By: btrahan

Subscribers: cburroughs, epriestley

Differential Revision: https://secure.phabricator.com/D10766

Details

Committed
epriestley <git@epriestley.com>Oct 31 2014, 17:27
Pushed
aubortJan 31 2017, 17:16
Parents
rPH915d9a52f085: Minor, fix bin/storage adjust for new column
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH914b8bb32cd7: Fix daemon task queue to respect task priority (authored by epriestley <git@epriestley.com>).Oct 31 2014, 17:27