-
Task
-
Resolution: Done
-
Neutral
-
None
-
None
-
-
Empty show more show less
-
Empty show more show less
-
Yes
-
Yes
-
Yes
As discovered in MAGNOLIA-9184, we need access to the RepositoryContext in order to register Jackrabbit's default MBeans with the MBean server.
Implementation notes
There's currently two proposals on how this could be implemented, both with their own pros and cons:
- PR1 specializes ProviderImpl into one that uses a different mechanism to get the repository, which provides access to the RepositoryContext.
- PR2 factors repository creation out into a RepositoryFactory, which provide the backward compatible implementation registering the repository with JNDI and the new one only providing access to the RepositoryContext.
1. is less risky as it does not touch the existing design and implementation much but it introduces some technical debt as we need to copy and slightly change a few classes from Jackrabbit (BindableMgnlRepository, BindableRepositoryFactory, RegistryHelper).
2. is more risky and reveals some tests that need fixing. There is a potential ripple effect of this causing similar test failures in other modules. Conceptually the approach is cleaner and introduces less technical debt.
- depends upon
-
MGNLUI-8612 FakeMagnoliaUI should register the default RepositoryManager with the compontent provider
- Closed
- is depended upon by
-
MAGNOLIA-9181 Expose EventListenerMBean for monitoring
- Closed
-
MAGNOLIA-9183 Expose QueryStatManagerMBean for monitoring
- Closed
-
MAGNOLIA-9184 Expose RepositoryStatistics for monitoring
- Closed