[SLING-5424] MBeanServer reference in MetricServiceImpl should be made optional

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Commons Metrics 1.0.0
• Fix Version/s: Commons Metrics 1.2.0
• Component/s: Commons
• Assignee: Chetan Mehrotra
• Reporter: Chetan Mehrotra
• Created: 12/Jan/16 10:02
• Updated: 13/Dec/16 06:36
• Resolved: 13/Jan/16 04:59

Description

Currently MetricServiceImpl has a required dependency on MBeanServer. Technically this can be made optional and default Platform MBeanServer can be used. Further due to missing start on JMXReporter the MBean are not getting registered also

Sling Launchpad registers an MBeanServer by default which might not be the case in other OSGi setups. So making this optional would simplify using this bundle in other setup also

Also look into testcase logic per discussion in http://markmail.org/thread/mow4u6vn7mfnj7gy

Issue Links

• blocks: SLING-5965 Metrics and a Health-Check for Scheduler to detect long-running Quartz-Jobs