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
Unknown Object (File)
Sat, Jun 20, 6:53 PM
Unknown Object (File)
Apr 5 2026, 4:55 AM
Unknown Object (File)
Apr 3 2026, 6:27 PM
Unknown Object (File)
Mar 30 2026, 3:11 AM
Unknown Object (File)
Mar 6 2026, 11:24 AM
Unknown Object (File)
Jan 22 2026, 10:06 PM
Unknown Object (File)
Dec 5 2025, 1:04 PM
Unknown Object (File)
Dec 4 2025, 10:16 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