Page MenuHomePhabricator

Use PhabricatorCachedClassMapQuery when querying object PHID types
ClosedPublic

Authored by epriestley on Dec 6 2016, 12:35 PM.
Tags
None
Referenced Files
F18842011: D16997.id.diff
Tue, Oct 28, 12:52 PM
F18840114: D16997.diff
Mon, Oct 27, 9:20 PM
F18825703: D16997.id40889.diff
Thu, Oct 23, 11:41 PM
F18804494: D16997.id40900.diff
Sat, Oct 18, 8:53 AM
F18737622: D16997.id40889.diff
Oct 1 2025, 11:40 AM
F18710285: D16997.id.diff
Sep 29 2025, 1:53 AM
F18644867: D16997.diff
Sep 19 2025, 6:13 AM
F18588422: D16997.id40900.diff
Sep 12 2025, 12:16 AM
Subscribers
None

Details

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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Use PhabricatorCachedClassMapQuery when querying object PHID types.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 6 2016, 4:21 PM