Cleanup "Failed to Unregister the MBean" error messages in logs, as they are 
not always errors
----------------------------------------------------------------------------------------------

                 Key: DSRV-15
                 URL: https://jira.duraspace.org/browse/DSRV-15
             Project: DSpace Services Module
          Issue Type: Improvement
          Components: kernel
            Reporter: Tim Donohue
            Priority: Major
             Fix For: 2.0.4
         Attachments: DSpaceKernelManager.patch

When the DSpace Kernel is shutdown (DSpaceKernelImpl.destroy()), it will 
occasionally report the following to the DSpace Logs:

2011-06-15 06:36:13,681 ERROR org.dspace.kernel.DSpaceKernelManager @ WARN 
Failed to unregister the MBean: 
org.dspace:name=7b509d98-8e23-4a6f-b518-3141009e8529,type=DSpaceKernel 

To me this error message has two main issues:
(1) I don't believe it's always in fact an error, despite the fact that it is 
reported as such in the logs
(2) The message itself gives no useful information to DSpace Users (or 
developers) as to why the MBean may not have been unregistered properly 

This error handling in the DSpaceKernelManager should be cleaned up and 
improved so that it provides more useful error/warning messages.

Here's the location of this error handling in DSpaceKernelManager:
https://fisheye3.atlassian.com/browse/dspace/modules/dspace-services/trunk/api/src/main/java/org/dspace/kernel/DSpaceKernelManager.java?hb=true#to183

I think the proper way to fix this would be to actually determine which form of 
error was thrown, and handle each properly. See the proposed 
DSpaceKernelManager.patch.

I'd appreciate it if someone can review these changes. If they seem reasonable, 
hopefully we can release this fix as part of version 2.0.4 (in DSpace 1.8.0)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://jira.duraspace.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security 
threats, fraudulent activity, and more. Splunk takes this data and makes 
sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2d-c2
_______________________________________________
Dspace-devel mailing list
Dspace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dspace-devel

Reply via email to