Page MenuHomePhabricator

In Phage, don't sit in a loop once we've read all messages from an agent
ClosedPublic

Authored by epriestley on Feb 19 2017, 3:04 PM.
Tags
None
Referenced Files
F15411546: D17387.diff
Wed, Mar 19, 9:26 AM
F15399595: D17387.id41810.diff
Mon, Mar 17, 5:23 AM
F15393985: D17387.diff
Sat, Mar 15, 11:18 PM
F15374776: D17387.diff
Wed, Mar 12, 8:17 PM
F15335339: D17387.diff
Sat, Mar 8, 3:06 PM
F15311740: D17387.diff
Thu, Mar 6, 10:37 AM
F15283351: D17387.diff
Tue, Mar 4, 6:59 AM
Unknown Object (File)
Fri, Feb 21, 3:12 PM
Subscribers
None

Details

Summary

Ref T2794. The logic is intended to read "loop until we're done processing messages", but is currently missing a break;. This doesn't affect behavior yet, just makes us eat way too much CPU.

Test Plan

Saw time phage remote --hosts db001-16 status drop from 99% CPU time to <10%.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable