You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ra...@apache.org on 2009/05/14 17:22:24 UTC

svn commit: r774817 - /qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java

Author: rajith
Date: Thu May 14 15:22:23 2009
New Revision: 774817

URL: http://svn.apache.org/viewvc?rev=774817&view=rev
Log:
This is related to QPID-1854
The sender.close is now surrounded with a try catch to handle the exception thrown by the IoSender.close().
The ideal situation would have been to pass false to the IoSender.close(boolean reportException) method which will then not throw the exception up the stack.
But that method is not exposed in the Sender interface.

Modified:
    qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java

Modified: qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java?rev=774817&r1=774816&r2=774817&view=diff
==============================================================================
--- qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java (original)
+++ qpid/trunk/qpid/java/common/src/main/java/org/apache/qpid/transport/Connection.java Thu May 14 15:22:23 2009
@@ -438,7 +438,14 @@
                 ssn.closed();
             }
 
-            sender.close();
+            try
+            {
+                sender.close();
+            }
+            catch(Exception e)
+            {
+                // ignore.
+            }
             sender = null;
             setState(CLOSED);
         }



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