Home GnuPG

Use PhabricatorCachedClassMapQuery when querying object PHID types
f8d6b6181eb4Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Use PhabricatorCachedClassMapQuery when querying object PHID types

Summary:
Ref T11954. When we query for Conduit tokens, we load the associated objects (users) by PHID.

Currently, querying objects by PHID requires us to load every PHIDType class, when we can know which specific classes we actually need (e.g., just UserPHIDType, if only user PHIDs are present in the query).

Use PhabricatorCachedClassMapQuery to reduce the number of classes we load on this pathway.

Test Plan:

  • Used ab -n100 to roughly measure a ~5% performance improvement?
  • This measurement feels a little flimsy but the XHProf profile is cleaner, at least.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11954

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

Details

Provenance
epriestley <git@epriestley.com>Authored on Dec 6 2016, 1:16 PM
Parents
rPHABbfbf75a8720c: Slightly modernize ConduitTokenQuery
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHABf8d6b6181eb4: Use PhabricatorCachedClassMapQuery when querying object PHID types (authored by epriestley <git@epriestley.com>).Dec 6 2016, 6:10 PM