Page MenuHomePhabricator

Do initial repository imports at a lower priority and finish importing commits before starting new ones
ClosedPublic

Authored by epriestley on Sep 21 2016, 10:38 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:59 AM
Unknown Object (File)
Tue, Apr 9, 11:20 AM
Unknown Object (File)
Tue, Mar 26, 3:03 AM
Unknown Object (File)
Feb 11 2024, 6:04 PM
Unknown Object (File)
Jan 29 2024, 2:15 AM
Unknown Object (File)
Jan 28 2024, 1:36 AM
Unknown Object (File)
Jan 18 2024, 7:26 AM
Unknown Object (File)
Dec 28 2023, 4:40 PM
Subscribers
None
Tokens
"Dat Boi" token, awarded by chad.

Details

Summary

Fixes T11677. This makes two minor adjustments to the repository import daemons:

  • The first step ("Message") now queues at a slightly-lower-than-default (for already-imported repositories) or very-low (for newly importing repositories) priority level.
  • The other steps now queue at "default" priority level. This is actually what they already did, but without this change their behavior would be to inherit the priority level of their parents.

This has two effects:

  • When adding new repositories to an existing install, they shouldn't block other things from happening anymore.
  • The daemons will tend to start one commit and run through all of its steps before starting another commit. This makes progress through the queue more even and predictable.
    • Before, they did ALL the message tasks, then ALL the change tasks, etc. This works fine but is confusing/uneven/less-predictable because each type of task takes a different amount of time.
Test Plan
  • Added a new repository.
  • Saw all of its "message" steps queue at priority 4000.
  • Saw followups queue at priority 2000.
  • Saw progress generally "finish what you started" -- go through the queue one commit at a time, instead of one type of task at a time.

Diff Detail

Repository
rP Phabricator
Branch
priority1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13804
Build 17839: Run Core Tests
Build 17838: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Do initial repository imports at a lower priority and finish importing commits before starting new ones.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 21 2016, 11:16 PM
This revision was automatically updated to reflect the committed changes.