Page MenuHomePhabricator

Allow archived tasks to be queried by object PHID and order by id
ClosedPublic

Authored by epriestley on Dec 30 2014, 5:47 PM.
Tags
None
Referenced Files
F13221030: D11089.diff
Sun, May 19, 2:13 AM
F13215362: D11089.id26627.diff
Fri, May 17, 4:50 PM
F13215035: D11089.id26628.diff
Fri, May 17, 2:26 PM
F13215034: D11089.id26618.diff
Fri, May 17, 2:26 PM
F13215032: D11089.id26627.diff
Fri, May 17, 2:26 PM
F13215031: D11089.id26625.diff
Fri, May 17, 2:26 PM
F13215030: D11089.id26623.diff
Fri, May 17, 2:26 PM
F13215029: D11089.id26626.diff
Fri, May 17, 2:26 PM
Subscribers

Details

Summary

Ref T5402.

Test Plan
  • Queried archived tasks.
  • Grepped for use sites and verified no other callsites are order-sensitive.

Diff Detail

Repository
rP Phabricator
Branch
daemons1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3403
Build 3410: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Allow archived tasks to be queried by object PHID and order by id.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Dec 30 2014, 6:00 PM
epriestley edited edge metadata.
  • Also copy the objectPHID into the archive task table when archiving a task.
  • And when unarchiving a task.
  • Also allow selecting leased tasks.
epriestley edited edge metadata.

This changed enough that it should probably get a second look in case I did anything goofy.

epriestley edited edge metadata.
  • Use "result set", which reads better than "results set".
  • Correct ordering/comments about the "leased" phase: we do it first, under the theory that those tasks are executing right now, then the unleased tasks are up next, then the expired tasks are further away.
  • Move all of the PHASE_LEASED clauses first, for consistency.
  • churn churn churn
  • When querying task information, reorder results by the original phase order.
btrahan edited edge metadata.

Still looks good. :D

This revision is now accepted and ready to land.Dec 30 2014, 8:53 PM
This revision was automatically updated to reflect the committed changes.