You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@chukwa.apache.org by bi...@apache.org on 2010/09/12 05:43:42 UTC

svn commit: r996244 - /incubator/chukwa/trunk/src/java/org/apache/hadoop/chukwa/datacollection/adaptor/jms/JMSAdaptor.java

Author: billgraham
Date: Sun Sep 12 03:43:42 2010
New Revision: 996244

URL: http://svn.apache.org/viewvc?rev=996244&view=rev
Log:
CHUKWA-518. JMSAdaptor does not shutdown properly

Modified:
    incubator/chukwa/trunk/src/java/org/apache/hadoop/chukwa/datacollection/adaptor/jms/JMSAdaptor.java

Modified: incubator/chukwa/trunk/src/java/org/apache/hadoop/chukwa/datacollection/adaptor/jms/JMSAdaptor.java
URL: http://svn.apache.org/viewvc/incubator/chukwa/trunk/src/java/org/apache/hadoop/chukwa/datacollection/adaptor/jms/JMSAdaptor.java?rev=996244&r1=996243&r2=996244&view=diff
==============================================================================
--- incubator/chukwa/trunk/src/java/org/apache/hadoop/chukwa/datacollection/adaptor/jms/JMSAdaptor.java (original)
+++ incubator/chukwa/trunk/src/java/org/apache/hadoop/chukwa/datacollection/adaptor/jms/JMSAdaptor.java Sun Sep 12 03:43:42 2010
@@ -289,9 +289,11 @@ public class JMSAdaptor extends Abstract
   public long shutdown(AdaptorShutdownPolicy shutdownPolicy)
       throws AdaptorException {
     try {
-      connection.stop();
+      connection.close();
 
-    } catch(Exception e) {}
+    } catch(Exception e) {
+      log.error("Exception closing JMS connection.", e);
+    }
 
     return bytesReceived;
   }