omit needless namespace refs; use nullopt
test metrics::intervals
cosmetic tweaks - const, indent
use synchronized<> to keep thread list