I think the idea is good, but I have 2 questions/comments:
1. The monitor method is documented to "get" metrics, but it takes a
`Metrics` instance. Either the method name/signature needs to be adjusted
or the documentation.
2. Instead of exposing the Metrics class, which is currently not exposed, I
think we should consider introducing an interface with the methods we want
to expose public.
On Mon, May 18, 2020 at 10:47 AM Rajini Sivaram <[EMAIL PROTECTED]>