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/09/12 20:33:02 UTC
svn commit: r1808163 -
/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/HttpPoolEntry.java
Author: pmouawad
Date: Tue Sep 12 20:33:02 2017
New Revision: 1808163
URL: http://svn.apache.org/viewvc?rev=1808163&view=rev
Log:
Comment on false-positive issue reported by antonio
Modified:
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/HttpPoolEntry.java
Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/HttpPoolEntry.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/HttpPoolEntry.java?rev=1808163&r1=1808162&r2=1808163&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/HttpPoolEntry.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/hc/HttpPoolEntry.java Tue Sep 12 20:33:02 2017
@@ -49,6 +49,11 @@ public class HttpPoolEntry extends PoolE
}
@Override
+ @SuppressWarnings("UnsynchronizedOverridesSynchronized")
+ /**
+ * No synchronization added as the override is just for logging.
+ * Adding synhronized within logging might introduce too much locking
+ */
public boolean isExpired(final long now) {
final boolean expired = super.isExpired(now);
if (expired && this.log.isDebugEnabled()) {