Page MenuHomePhabricator

Support sampling subprocess execution in Multimeter
ClosedPublic

Authored by epriestley on May 3 2015, 5:24 PM.
Tags
None
Referenced Files
F14482862: D12675.diff
Mon, Dec 30, 12:07 PM
Unknown Object (File)
Sat, Dec 21, 3:50 PM
Unknown Object (File)
Nov 27 2024, 9:22 PM
Unknown Object (File)
Nov 20 2024, 3:34 PM
Unknown Object (File)
Nov 15 2024, 5:56 PM
Unknown Object (File)
Nov 12 2024, 2:38 AM
Unknown Object (File)
Nov 8 2024, 2:05 PM
Unknown Object (File)
Oct 29 2024, 3:05 PM
Subscribers

Details

Summary

Ref T6930. Add subprocess sampling/labeling.

Test Plan

(This table includes some buggy test labels.)

Screen Shot 2015-05-03 at 10.23.57 AM.png (1×1 px, 249 KB)

Diff Detail

Repository
rP Phabricator
Branch
mm6
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5701
Build 5720: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Support sampling subprocess execution in Multimeter.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
btrahan added inline comments.
src/applications/multimeter/data/MultimeterControl.php
236–239

Seems like a good idea for now, though I predict we'll at least need to grow this list a bit over time.

This revision is now accepted and ready to land.May 4 2015, 4:54 PM
src/applications/multimeter/data/MultimeterControl.php
236–239

maybe we just end up with some conf dict that gets merged in? we can then have admins mess with the conf if we're looking for specific performance issues to get sampled in all their glory.

Since we don't actually use the table until the very end of the request, we could let third-party applications do this sort of thing:

$multimeter = ...
$multimeter->addCommandAbbreviationConfig(...);

...once we run into it in the wild.

This revision was automatically updated to reflect the committed changes.