Page MenuHomePhabricator

When many commits are discovered at once, import them at lower priority

Authored by epriestley on Aug 12 2019, 7:09 PM.



Ref T13369. See that task for discussion.

When the discovery daemon finds more than 64 commits to import, demote the worker queue priority of the resulting tasks.

Test Plan
  • Pushed one commit, ran bin/repository discover --verbose --trace ..., saw commit import with "at normal priority" message and priority 2500 ("PRIORITY_COMMIT").
  • Pushed 3 commits, set threshold to 3, ran bin/repository discover ..., saw commist import with "at lower priority" message and priority 4000 ("PRIORITY_IMPORT").

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Aug 12 2019, 7:09 PM
epriestley requested review of this revision.Aug 12 2019, 7:11 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 12 2019, 7:59 PM
This revision was automatically updated to reflect the committed changes.