HomePhabricator

Use LiskDAOSet's loadRelativeEdges with ReleephRequest

Description

Use LiskDAOSet's loadRelativeEdges with ReleephRequest

Summary:
ReleephRequests contain the PHID for a PhabricatorRepositoryCommit, and commits have an edge to a DifferentialRevision.

Commits are loaded with the loadOneRelative() method that loads the commits for every ReleephRequest in a LiskDAOSet, but the edges are loaded indivdually. A page with N RQs on it makes one DB query for the commits, but N queries for the TYPE_COMMIT_HAS_DREV edges.

This diff uses loadRelativeEdges instead to load the edges all in one query.

Test Plan: {F42290}

Reviewers: wez, epriestley

Reviewed By: epriestley

CC: epriestley, vrana, aran

Maniphest Tasks: T2714

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

Details

Provenance
Edward Speyer <edward@fb.com>Authored on May 1 2013, 9:22 AM
Reviewer
epriestley
Differential Revision
Restricted Differential Revision
Parents
rP30b15e094c59: Load commits affected by revert language in Diffusion message parser
Branches
Unknown
Tags
Unknown
Tasks
T2714: Move Releeph to Beta