You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2017/02/24 17:24:00 UTC

svn commit: r1784305 - in /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc: JMeterPoolingClientConnectionManager.java ManagedClientConnectionImpl.java

Author: pmouawad
Date: Fri Feb 24 17:24:00 2017
New Revision: 1784305

URL: http://svn.apache.org/viewvc?rev=1784305&view=rev
Log:
Fix Sonar errors

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/JMeterPoolingClientConnectionManager.java
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/ManagedClientConnectionImpl.java

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/JMeterPoolingClientConnectionManager.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/JMeterPoolingClientConnectionManager.java?rev=1784305&r1=1784304&r2=1784305&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/JMeterPoolingClientConnectionManager.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/JMeterPoolingClientConnectionManager.java Fri Feb 24 17:24:00 2017
@@ -253,7 +253,7 @@ public class JMeterPoolingClientConnecti
             // Should never happen
             throw new InterruptedException();
         } catch (final TimeoutException ex) {
-            throw new ConnectionPoolTimeoutException("Timeout waiting for connection from pool");
+            throw new ConnectionPoolTimeoutException("Timeout waiting for connection from pool, message:"+ex.getMessage()); // NOSONAR 
         }
     }
     @Override

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/ManagedClientConnectionImpl.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/ManagedClientConnectionImpl.java?rev=1784305&r1=1784304&r2=1784305&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/ManagedClientConnectionImpl.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/ManagedClientConnectionImpl.java Fri Feb 24 17:24:00 2017
@@ -476,7 +476,8 @@ public class ManagedClientConnectionImpl
             final OperatedClientConnection conn = this.poolEntry.getConnection();
             try {
                 conn.shutdown();
-            } catch (final IOException ignore) {
+            } catch (final IOException ignore) { // NOSONAR It ok to ignore
+                // NOOP
             }
             this.manager.releaseConnection(this, this.duration, TimeUnit.MILLISECONDS);
             this.poolEntry = null;