[
https://issues.apache.org/jira/browse/SLING-5424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Chetan Mehrotra updated SLING-5424:
-----------------------------------
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
was:
Currently {{MetricServiceImpl}} has a required dependency on MBeanServer.
Technically this can be made optional and default Platform MBeanServer can be
used.
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
> MBeanServer reference in MetricServiceImpl should be made optional
> ------------------------------------------------------------------
>
> Key: SLING-5424
> URL: https://issues.apache.org/jira/browse/SLING-5424
> Project: Sling
> Issue Type: Bug
> Components: Commons
> Affects Versions: Commons Metrics 1.0.0
> Reporter: Chetan Mehrotra
> Assignee: Chetan Mehrotra
> Priority: Minor
> Fix For: Commons Metrics 1.0.2
>
>
> 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
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
