HomePhorge

Filter archived packages out of the "controlling packages" query earlier

Description

Filter archived packages out of the "controlling packages" query earlier

Summary:
Ref T12319. Currently, we end up filtering archived packages out once for each path. This shows up on a profile from an install as meaningfully expensive:

https://secure.phabricator.com/xhprof/profile/PHID-FILE-7kmpevyr22aih4s2vyln/?symbol=PhabricatorOwnersPackage::isArchived

Instead, filter them out before we do any work.

Test Plan:
Viewed a revision, still saw packages.

{F3425553}

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12319

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

Details

Provenance
epriestleyAuthored on Feb 27 2017, 12:08 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP05377bea193c: Add an avatar builtin file generator
Branches
Unknown
Tags
Unknown

Event Timeline