qpid-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ai...@apache.org
Subject svn commit: r748591 - in /qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration: ServerConfiguration.java management/ConfigurationManagement.java
Date Fri, 27 Feb 2009 16:57:22 GMT
Author: aidan
Date: Fri Feb 27 16:57:22 2009
New Revision: 748591

URL: http://svn.apache.org/viewvc?rev=748591&view=rev
Log:
QPID-1699: fix mbean annotation, log failure to reload file. Address review comments from
robbie.

Modified:
    qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ServerConfiguration.java
    qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/management/ConfigurationManagement.java

Modified: qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ServerConfiguration.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ServerConfiguration.java?rev=748591&r1=748590&r2=748591&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ServerConfiguration.java
(original)
+++ qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/ServerConfiguration.java
Fri Feb 27 16:57:22 2009
@@ -27,8 +27,6 @@
 import java.util.Map;
 import java.util.Map.Entry;
 
-import javax.management.NotCompliantMBeanException;
-
 import org.apache.commons.configuration.CompositeConfiguration;
 import org.apache.commons.configuration.Configuration;
 import org.apache.commons.configuration.ConfigurationException;
@@ -39,6 +37,9 @@
 import org.apache.qpid.server.registry.ApplicationRegistry;
 import org.apache.qpid.server.virtualhost.VirtualHost;
 import org.apache.qpid.server.virtualhost.VirtualHostRegistry;
+import org.apache.qpid.tools.messagestore.MessageStoreTool;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import sun.misc.Signal;
 import sun.misc.SignalHandler;
@@ -62,8 +63,11 @@
     private SecurityConfiguration _securityConfiguration = null;
 
     private File _configFile;
+    
+    private Logger _log = LoggerFactory.getLogger(this.getClass());
 
     private ConfigurationManagementMBean _mbean;
+    
 
     // Map of environment variables to config items
     private static final Map<String, String> envVarMap = new HashMap<String, String>();
@@ -206,7 +210,7 @@
         }
         catch (ConfigurationException e)
         {
-            // Not much we can do about it really. 
+             _log.error("Could not reload configuration file", e);
         }        
     }
 

Modified: qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/management/ConfigurationManagement.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/management/ConfigurationManagement.java?rev=748591&r1=748590&r2=748591&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/management/ConfigurationManagement.java
(original)
+++ qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/configuration/management/ConfigurationManagement.java
Fri Feb 27 16:57:22 2009
@@ -20,8 +20,10 @@
  */
 package org.apache.qpid.server.configuration.management;
 
+import javax.management.MBeanOperationInfo;
+
 import org.apache.commons.configuration.ConfigurationException;
-import org.apache.qpid.server.management.MBeanAttribute;
+import org.apache.qpid.server.management.MBeanOperation;
 
 public interface ConfigurationManagement
 {
@@ -33,8 +35,9 @@
      * Reload the 
      * @throws ConfigurationException 
      */
-    @MBeanAttribute(name="reloadSecurityConfiguration", 
-                    description = "Force a reload of the security configuration sections")
+    @MBeanOperation(name="reloadSecurityConfiguration", 
+                    description = "Force a reload of the security configuration sections",
+                    impact = MBeanOperationInfo.ACTION)
     void reloadSecurityConfiguration() throws ConfigurationException;
     
 }



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org


Mime
View raw message