Homestyx hydra
Diffusion hydra 8e0a975e3f0f

Remove calls to DifferentialRevision::loadRelations from commit workers

Description

Remove calls to DifferentialRevision::loadRelations from commit workers

Summary:
Remove ocurrences of loadRelations in workers.

One was simply unnecesary, no subsequent call to getReviewers or getCCPHIDs was made.
The other was replaced with the nicer DifferentialRevisionQuery using needRelations and needReviewerStatus (for future upgrade).

Test Plan:
Land a revision into a tracked repository and check the parser worker attached the commit correctly.
For the owners worker I just checked it didn't crash into a hundred tiny pieces.

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

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

Details

Provenance
Juan Pablo CivileAuthored on Jul 15 2013, 9:39 PM
epriestleyCommitted on Jul 15 2013, 9:39 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:ee9fac5c8fb7: Use DifferentialRevisionQuery in differential controllers
Branches
Unknown
Tags
Unknown

Event Timeline