Though `go dms.Serve()` would be okay, I believe the "Leave concurrency up to the caller" section is not targeted at this kind of case. In particular, the `DiskMetricStore` cannot be used without the loop started, and so makes sense to me to happen in the constructor.