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

	Unknown Object (File)
	Wed, Apr 17, 3:09 PM

	Unknown Object (File)
	Thu, Apr 11, 7:18 AM

	Unknown Object (File)
	Sat, Apr 6, 11:31 AM

	Unknown Object (File)
	Wed, Apr 3, 2:51 PM

	Unknown Object (File)
	Mon, Apr 1, 4:24 AM

	Unknown Object (File)
	Thu, Mar 28, 10:06 PM

	Unknown Object (File)
	Jan 23 2024, 2:59 PM

	Unknown Object (File)
	Jan 16 2024, 9:15 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...