Page MenuHomePhabricator

Trigger "import" heuristics for imported repositories
Closed, ResolvedPublic

Description

Since D11827 we have an "import" heuristic which triggers on pushes to hosted repositories: if you push more than 7 commits to an imported, empty repository, we kick it back into importing mode.

This heuristic does not activate for imported repositories because we don't know where the boundaries of pushes are.

We could at least try to do the same thing with imported repositories, although it is less reliable and somewhat race-prone: if we discover more than 7 commits in an imported, empty repository, kick it back into importing mode. It's possible that the parsers will get ahead of discovery and we'll lose the mode before import finishes, but this should be a difficult race for the discovery daemon to lose since the import daemons need to jump through so many steps.

Revisions and Commits

Restricted Differential Revision

Event Timeline

epriestley raised the priority of this task from to Normal.
epriestley updated the task description. (Show Details)
epriestley added a project: Diffusion.
epriestley updated the task description. (Show Details)
epriestley added subscribers: epriestley, angie.
angie added a project: Restricted Project.Oct 13 2015, 12:47 AM
angie added a subscriber: jhurwitz.
eadler added a project: Restricted Project.Jan 9 2016, 1:02 AM
eadler moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.
epriestley edited projects, added Diffusion (v3); removed Diffusion.
epriestley moved this task from v3 to Backlog on the Diffusion board.
epriestley edited projects, added Diffusion; removed Diffusion (v3).
eadler moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.Feb 18 2016, 6:33 PM
epriestley added a revision: Restricted Differential Revision.May 10 2016, 2:05 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.May 11 2016, 1:36 PM
epriestley added a commit: Restricted Diffusion Commit.