[jboss-cvs] JBossAS SVN: r109248 - branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers.

[jboss-cvs-commits at lists.jboss.org]

Author: miclark
Date: 2010-11-10 15:17:10 -0500 (Wed, 10 Nov 2010)
New Revision: 109248

Modified:
   branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatDeployerMBean.java
   branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatService.java
Log:
JBPAPP-4960: Added missing JMX Notifications for tomcat connectors

Modified: branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatDeployerMBean.java
===================================================================
--- branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatDeployerMBean.java	2010-11-10 20:16:58 UTC (rev 109247)
+++ branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatDeployerMBean.java	2010-11-10 20:17:10 UTC (rev 109248)
@@ -39,6 +39,9 @@
    /** JMX notification type to signal after-start connector event */
    public final String TOMCAT_CONNECTORS_STARTED = "jboss.tomcat.connectors.started";
 
+   /** JMX notification type to signal after-start connector event */
+   public final String TOMCAT_CONNECTORS_STOPPED = "jboss.tomcat.connectors.stopped";
+
    /**
     * @return the jmx domain for the tomcat management mbeans
     */

Modified: branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatService.java
===================================================================
--- branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatService.java	2010-11-10 20:16:58 UTC (rev 109247)
+++ branches/JBPAPP_5_1/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatService.java	2010-11-10 20:17:10 UTC (rev 109248)
@@ -622,11 +622,11 @@
          Lifecycle lc = connectors[n];
          lc.start();
       }
-      /*
-       * TODO: // Notify listeners that connectors have started processing requests sendNotification(new
-       * Notification(TOMCAT_CONNECTORS_STARTED, this, getNextNotificationSequenceNumber()));
-       */
+
       connectorsRunning = true;
+
+      // Notify listeners that connectors have stqred processing requests (JBPAPP-4960)
+      sendNotification(new Notification(TOMCAT_CONNECTORS_STARTED, this, getNextNotificationSequenceNumber()));
    }
 
    public void stopConnectors() throws Exception
@@ -636,6 +636,9 @@
       if (!connectorsRunning)
          return;
 
+      // Notify listeners that connectors have stopped processing requests (JBPAPP-4960)
+      sendNotification(new Notification(TOMCAT_CONNECTORS_STOPPED, this, getNextNotificationSequenceNumber()));
+
       MBeanServer server = super.getServer();
       ObjectName service = new ObjectName(tomcatDeployer.getDomain() + ":type=Service,serviceName=jboss.web");
       Object[] args = {};