Paths

Table of Contentst

Differential D17400

Use ApplicationSearch in XHProf
ClosedPublic
Actions

Authored by epriestley on Feb 23 2017, 5:59 PM.

Tags

None

Referenced Files

	F18761684: D17400.id41839.diff
	Mon, Oct 6, 3:41 PM

	F18716312: D17400.id41844.diff
	Mon, Sep 29, 11:45 AM

	F18665446: D17400.diff
	Sep 24 2025, 2:12 AM

	F18653499: D17400.diff
	Sep 21 2025, 2:04 PM

	F18606318: D17400.diff
	Sep 13 2025, 10:05 PM

	F18585997: D17400.id41839.diff
	Sep 11 2025, 5:04 PM

	F18560229: D17400.id41844.diff
	Sep 8 2025, 10:48 PM

	F18461419: D17400.id41839.diff
	Sep 1 2025, 8:29 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12297: Make Conduit API calls on `admin.phacility.com` reasonably easy to profile

Commits

rP425470227193: Use ApplicationSearch in XHProf

Summary

Ref T12297. This slightly modernizes the XHProf UI. Not included here:

Some of the code acts like samples have PHIDs, but they currently do not. I plan to add them in the next change.
I've intentionally left the actual list untouched for now -- it has some old/buggy code (like flag-6 is no longer an icon) that I'll fix in a future change.

Test Plan

Screen Shot 2017-02-23 at 9.56.14 AM.png (1×1 px, 308 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 23 2017, 5:59 PM

Harbormaster completed remote builds in B15736: Diff 41839.Feb 23 2017, 5:59 PM

Harbormaster completed remote builds in B15736: Diff 41839.

chad accepted this revision.Feb 23 2017, 6:17 PM

This revision is now accepted and ready to land.Feb 23 2017, 6:17 PM

Closed by commit rP425470227193: Use ApplicationSearch in XHProf (authored by epriestley, committed by epriestley). · Explain WhyFeb 23 2017, 7:16 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

9 lines

applications/

xhprof/

controller/

PhabricatorXHProfSampleListController.php

90 lines

query/

PhabricatorXHProfSampleQuery.php

51 lines

PhabricatorXHProfSampleSearchEngine.php

97 lines

storage/

PhabricatorXHProfSample.php

25 lines

Diff 41844

src/__phutil_library_map__.php

Loading...

src/applications/xhprof/controller/PhabricatorXHProfSampleListController.php

Loading...

src/applications/xhprof/query/PhabricatorXHProfSampleQuery.php

Loading...

src/applications/xhprof/query/PhabricatorXHProfSampleSearchEngine.php

Loading...

src/applications/xhprof/storage/PhabricatorXHProfSample.php

Loading...