[
https://issues.apache.org/jira/browse/SLING-2742?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13585727#comment-13585727
]
Felix Meschberger commented on SLING-2742:
------------------------------------------
I like the idea very much, but I am not comfortable with the patch:
* I don't think we should expose the RepositoryStatistics as a service. These
statistics pertain to a repository and should be related to the repository
only, thus retrieved from the appropriate repository.
* The MBean interface should define the MBean API for the benefit of JMX
Management Agents. Otherwise the MBean is useless. Probably having the MBean
just extend the Jackrabbit RepositoryStatistics would suffice it -- probably
not, though, due to the non-JMX API used by RepositoryStatistics.
* The MBean should probably be registered when the repository instance is
registered as a service: Probably SlingServerRepository should overwrite the
registerService and unregisterService methods to register/unregister the MBean
service appropriately. This also makes the setup easier because there is no
reflection required.
> Expose RepositoryStatistics as a JMX Bean
> ------------------------------------------
>
> Key: SLING-2742
> URL: https://issues.apache.org/jira/browse/SLING-2742
> Project: Sling
> Issue Type: Bug
> Components: JCR
> Affects Versions: JCR Jackrabbit Server 2.1.0
> Reporter: Ian Boston
> Attachments: SLING-2742.patch
>
>
> Jackrabbit comes with reasonably extensive stats which should be exposed via
> JMX for monitoring. Searching the current code base indicates that this has
> not been done already.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
