Homec4science

For now, hard limit task graph at 100 nodes

Authored by epriestley <git@epriestley.com> on Jul 14 2016, 06:07.

Description

For now, hard limit task graph at 100 nodes

Summary:
Ref T4788. One install has some particularly impressive task graphs which are thousands of nodes large.

The current graph is pretty broken in these cases. For now, just render a "too big to show" message. In the future, I'd plan to finesse this (e.g., show parents/children, show links to parents/children, etc).

Test Plan:

  • Viewed a normal task.
  • Set limit to 3, viewed a task with graph size 6, saw an error message.
  • Viewed a revision stack graph (unaffected).

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T4788

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

Details

Committed
epriestley <git@epriestley.com>Jul 14 2016, 06:15
Pushed
aubortJan 31 2017, 17:16
Parents
rPH6eaa9faec77b: Provide default view and edit policies in Calendar, plus "Event Host" and…
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH2a1b8ce85bef: For now, hard limit task graph at 100 nodes (authored by epriestley <git@epriestley.com>).Jul 14 2016, 06:15