Homec4science

Allow worker tasks to have priorities

Authored by Joshua Spence <josh@joshuaspence.com> on Jul 11 2014, 19:02.

Description

Allow worker tasks to have priorities

Summary: Fixes T5336. Currently, PhabricatorWorkerLeaseQuery is basically FIFO. It makes more sense for the queue to be a priority-queue, and to assign higher priorities to alerts (email and SMS).

Test Plan: Created dummy tasks in the queue (with different priorities). Verified that the priority field was set correctly in the DB and that the priority was shown on the /daemon/ page. Started a PhabricatorTaskmasterDaemon and verified that the higher priority tasks were executed before lower priority tasks.

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: epriestley, Korvin

Maniphest Tasks: T5336

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

Details

Committed
Joshua Spence <josh@joshuaspence.com>Jul 11 2014, 19:02
Pushed
aubortJan 31 2017, 17:16
Parents
rPH66a3abe058a7: Make it possible to configure Elasticsearch index name
Branches
Unknown
Tags
Unknown

Event Timeline

Joshua Spence <josh@joshuaspence.com> committed rPH9a679bf3746c: Allow worker tasks to have priorities (authored by Joshua Spence <josh@joshuaspence.com>).Jul 11 2014, 19:02