You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ie...@apache.org on 2016/09/21 13:14:52 UTC

svn commit: r1761725 - /sling/trunk/contrib/commons/mom/jms/src/main/java/org/apache/sling/jms/JMSQueueManager.java

Author: ieb
Date: Wed Sep 21 13:14:51 2016
New Revision: 1761725

URL: http://svn.apache.org/viewvc?rev=1761725&view=rev
Log:
fixing potential race condition causing a NPE inside the JMS provider, failing tests intermittently

Modified:
    sling/trunk/contrib/commons/mom/jms/src/main/java/org/apache/sling/jms/JMSQueueManager.java

Modified: sling/trunk/contrib/commons/mom/jms/src/main/java/org/apache/sling/jms/JMSQueueManager.java
URL: http://svn.apache.org/viewvc/sling/trunk/contrib/commons/mom/jms/src/main/java/org/apache/sling/jms/JMSQueueManager.java?rev=1761725&r1=1761724&r2=1761725&view=diff
==============================================================================
--- sling/trunk/contrib/commons/mom/jms/src/main/java/org/apache/sling/jms/JMSQueueManager.java (original)
+++ sling/trunk/contrib/commons/mom/jms/src/main/java/org/apache/sling/jms/JMSQueueManager.java Wed Sep 21 13:14:51 2016
@@ -175,7 +175,11 @@ public class JMSQueueManager implements
         }
 
         public void close() {
-            session.close();
+            try {
+                session.close();
+            } catch ( Exception e ) {
+                LOGGER.debug("Exception when closing session.",e);
+            }
         }
     }