MetricsService composed of TimerService instead of inheriting

This makes testing easier. Also, passing reactor and log to the instance
for better testing
test fix after log change
removed type directive - thanks @radix
Merge pull request #744 from rackerlabs/metrics_continue_on_err

Metrics service continues on error