-
Task
-
Resolution: Unresolved
-
Minor
-
None
-
None
-
None
The ObjStoreBrowser (https://github.com/jbosstm/narayana/blob/main/ArjunaCore/arjuna/classes/com/arjuna/ats/arjuna/tools/osb/mbean/ObjStoreBrowser.java) class is a singleton so it is preferable to use a static method to retrieve the instance (getInstance method). Please change the contructor to private to align at the singleton pattern.
Please consider removing the constructor accepting the logDir parameter because the store directory should be set as 'arjPropertyManager.getObjectStoreEnvironmentBean().setObjectStoreDir("logDir");'
Please note that this is an API change and needs a major release.
Additionally the ObjectStoreEnvironmentBean:::setObjectStoreDir could be seen as slightly different use case to what the OSB would use it for: possibly an OSB user wants to connect to a different object store than the one that the transaction manager in memory is configured to use. Since that case is probably very rare and (I think) misleading it is better relying on the field 'ObjectStoreEnvironmentBean.setObjectStoreDir'. That being said, we would want a release note for it so we make sure the impact is clear.
- relates to
-
JBTM-3973 ObjectStoreBrowser doesn't allow to register external OSBTypeHandler
- Resolved