HomePhabricator

Don't load the entire graph for tasks

Description

Don't load the entire graph for tasks

Summary:
Ref T4788. As it turns out, our tasks are very tightly connected.

Instead of loading every parent/child task, then every parent/child of those tasks, etc., etc., only load tasks in the "same direction" that we're already heading.

For example, we load children of children, but not parents of children. And we load parents of parents, but not children of parents.

Basically we only go "up" and "down" now, but not "out" as much. This should reduce the gigantic multiple-thousand-node graphs currently shown in the UI.

I still discover the whole graph for revisiosn, because I think it's probably more useful and always much smaller. That might need adjustment too, though.

Test Plan: Seems fine locally??

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T4788

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

Details

Provenance
epriestleyAuthored on Jul 1 2016, 6:37 PM
epriestleyPushed on Jul 1 2016, 6:43 PM
Reviewer
chad
Differential Revision
D16218: Don't load the entire graph for tasks
Parents
rPf26374241ab1: Make Phame Header and Profile Image Transactional
Branches
Unknown
Tags
Unknown
Tasks
T4788: Allow "Edit Dependencies" both ways (blocking and depending)
Build Status
Buildable 12893
Build 16444: Run Core Tests