[jboss-cvs] JBossAS SVN: r111303 - branches/JBPAPP_5_1_0_Final_JBPAPP-6476/tomcat/src/main/org/jboss/web/tomcat/service/deployers.

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

Author: bmaxwell
Date: 2011-05-06 14:24:08 -0400 (Fri, 06 May 2011)
New Revision: 111303

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

Modified: branches/JBPAPP_5_1_0_Final_JBPAPP-6476/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatDeployerMBean.java
===================================================================
--- branches/JBPAPP_5_1_0_Final_JBPAPP-6476/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatDeployerMBean.java	2011-05-06 17:40:31 UTC (rev 111302)
+++ branches/JBPAPP_5_1_0_Final_JBPAPP-6476/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatDeployerMBean.java	2011-05-06 18:24:08 UTC (rev 111303)
@@ -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_0_Final_JBPAPP-6476/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatService.java
===================================================================
--- branches/JBPAPP_5_1_0_Final_JBPAPP-6476/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatService.java	2011-05-06 17:40:31 UTC (rev 111302)
+++ branches/JBPAPP_5_1_0_Final_JBPAPP-6476/tomcat/src/main/org/jboss/web/tomcat/service/deployers/TomcatService.java	2011-05-06 18:24:08 UTC (rev 111303)
@@ -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 = {};