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

	F15572050: D17400.id.diff
	Mon, May 5, 11:51 AM

	F15444325: D17400.id41844.diff
	Mar 27 2025, 9:03 AM

	F15435659: D17400.id41839.diff
	Mar 25 2025, 9:34 AM

	F15425204: D17400.diff
	Mar 23 2025, 3:10 AM

	F15391438: D17400.id41844.diff
	Mar 15 2025, 9:52 AM

	F15300371: D17400.diff
	Mar 5 2025, 4:42 PM

	Unknown Object (File)
	Feb 26 2025, 7:28 PM

	Unknown Object (File)
	Feb 26 2025, 7:23 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...