Hi, I had a look at SLING-5965, this looks useful but raised two questions, mostly because I'm not yet familiar with the new o.a.s.commons.metrics that Chetan introduced.
1) Is it generally ok for Gauge.getValue() (for example, as this is what this code uses) to have to do some work to produce a value, as opposed to accumulating values earlier. With the current SLING-5965 patch, getValue() takes more time the more jobs are present. The alternative might be to just store the N oldest job starting times. Creating load while retrieving metrics looks risky to me in general as it's when your system is in trouble that you might request those metrics the most. 2) Does the metrics library by default make all values available via JMX as well? -Bertrand
