[ 
https://issues.apache.org/jira/browse/SLING-5443?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15127788#comment-15127788
 ] 

Carsten Ziegeler commented on SLING-5443:
-----------------------------------------

I think having a service factory is good, having a global mapping is not very 
user friendly . It makes it hard for each bundle to provide it's own mapping - 
we could work with factory configurations, but that still seems be a lot of 
overhead
Instead, I suggest we simply use a bundle header for the mapping. If none is 
available, the bundle symbolic name is used, otherwise the value from that 
header is used instead.

> Review the naming conventions used by JMXReporter to register the Metric 
> MBeans
> -------------------------------------------------------------------------------
>
>                 Key: SLING-5443
>                 URL: https://issues.apache.org/jira/browse/SLING-5443
>             Project: Sling
>          Issue Type: Improvement
>          Components: Commons
>            Reporter: Chetan Mehrotra
>            Assignee: Chetan Mehrotra
>             Fix For: Commons Metrics 1.0.2
>
>
> With current setup the JMXReporter would register all metrics under 
> {{org.apache.sling}} domain. [~pfa...@adobe.com] suggested that this can 
> cause confusion as metrics registered by non Sling bundle would also show up 
> in Sling JMX domain. 
> We should customize the JMX ObjectName logic to account for bundle which has 
> registered the metrics. One approach that can be used is 
> # Expose the MetricService as a ServiceFactory
> # Have an OSGi config which provides a mapping between Bundle-SymbolicName 
> and JMX Domain name to use. It would be a regex expression. The metric though 
> MUST be unique across all metrics (irrespective of which bundle was used)
> {noformat}
> com.foo = com.foo.*
> org.apache.sling=org.apache.sling.*
> {noformat}
> Discussion thread on mailing list http://markmail.org/thread/sj6yvmyhgze6jn22



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to