ciechanowiec commented on PR #11: URL: https://github.com/apache/sling-org-apache-sling-jcr-oak-server/pull/11#issuecomment-2416971136
> Apologies for taking so long to review. Have you tried using [Oak.with(MBeanServer)](https://github.com/apache/jackrabbit-oak/blob/cae54edb04b092fc48fcb558aa910c4092119e2f/oak-core/src/main/java/org/apache/jackrabbit/oak/Oak.java#L559-L562) instead of adding extra code to handle this? @rombert, yes, I tried the method you suggested, specifically `Oak.with(MBeanServer)`. However, it doesn’t appear to solve the issue. The only relevant effect of `Oak.with(MBeanServer)` is that it allows specifying which `MBeanServer` will be used by the constructed repository. The missing MBeans still aren’t registered with any instance of the `MBeanServer`. Therefore, the code proposed in this pull request is still necessary, as it introduces the registration of the missing MBeans. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
