Page MenuHomePhabricator

Support sampling subprocess execution in Multimeter
ClosedPublic

Authored by epriestley on May 3 2015, 5:24 PM.
Tags
None
Referenced Files
F18838560: D12675.diff
Mon, Oct 27, 11:27 AM
F18830626: D12675.id.diff
Sat, Oct 25, 7:45 AM
F18829590: D12675.diff
Fri, Oct 24, 11:02 PM
F18782614: D12675.id.diff
Mon, Oct 13, 2:10 AM
F18648263: D12675.diff
Sep 19 2025, 10:48 PM
F18646536: D12675.id30440.diff
Sep 19 2025, 10:14 AM
F18578830: D12675.id30486.diff
Sep 10 2025, 11:08 PM
F18578826: D12675.id30440.diff
Sep 10 2025, 11: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.