jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: svn commit: r1402642 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
Date Mon, 29 Oct 2012 01:00:07 GMT
On 26 October 2012 20:58,  <pmouawad@apache.org> wrote:
> Author: pmouawad
> Date: Fri Oct 26 19:58:30 2012
> New Revision: 1402642
>
> URL: http://svn.apache.org/viewvc?rev=1402642&view=rev
> Log:
> Use same log level as test on log level

-1

The fix looks correct, but in fact the log.warn was intentional; it's
really the check of isDebugEnabled that is wrong - see my comment on
r1402574.

> Modified:
>     jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
>
> Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java?rev=1402642&r1=1402641&r2=1402642&view=diff
> ==============================================================================
> --- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
(original)
> +++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/HTTPSamplerBase.java
Fri Oct 26 19:58:30 2012
> @@ -754,7 +754,7 @@ public abstract class HTTPSamplerBase ex
>      public void setAuthManager(AuthManager value) {
>          AuthManager mgr = getAuthManager();
>          if (log.isDebugEnabled() && mgr != null) {
> -            log.warn("Existing AuthManager " + mgr.getName() + " superseded by " + value.getName());
> +            log.debug("Existing AuthManager " + mgr.getName() + " superseded by " +
value.getName());
>          }
>          setProperty(new TestElementProperty(AUTH_MANAGER, value));
>      }
>
>

Mime
View raw message