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: r1798401 - /jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContextService.java
Date Mon, 12 Jun 2017 11:38:37 GMT
On 11 June 2017 at 21:19,  <pmouawad@apache.org> wrote:
> Author: pmouawad
> Date: Sun Jun 11 20:19:54 2017
> New Revision: 1798401
>
> URL: http://svn.apache.org/viewvc?rev=1798401&view=rev
> Log:
> Allow to use variables ( from User Defined Variables only ) in all listeners in slave
mode
> Renamve method to make it clearer
>
> Bugzilla Id: 57962
>
> Modified:
>     jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContextService.java
>
> Modified: jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContextService.java
> URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContextService.java?rev=1798401&r1=1798400&r2=1798401&view=diff
> ==============================================================================
> --- jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContextService.java (original)
> +++ jmeter/trunk/src/core/org/apache/jmeter/threads/JMeterContextService.java Sun Jun
11 20:19:54 2017
> @@ -47,9 +47,8 @@ public final class JMeterContextService
>      //@GuardedGy("this")

=> GuardedBy

>      private static int totalThreads = 0;
>
> -    //@GuardedGy("this")

Why was this comment deleted?

>      private static JMeterVariables variables = null;
> -
> +
>
>      /**
>       * Private constructor to prevent instantiation.
> @@ -168,10 +167,11 @@ public final class JMeterContextService
>      }
>
>      /**
> -     * Get all variables accessible for JMeter client in a distributed test (exclusively
test plan and user defined variables )
> -     * @return variables available for JMeter client
> +     * Get all variables accessible for JMeter client in a distributed test
> +     * (only test plan and user defined variables)
> +     * @return {@link JMeterVariables} available for JMeter client
>       */
> -    public static JMeterVariables getClientVariable() {
> +    public static JMeterVariables getClientVariables() {
>          return variables;
>      }
>
> @@ -179,9 +179,9 @@ public final class JMeterContextService
>       * Set variables for JMeter client in a distributed test
>       * @param var {@link JMeterVariables}
>       */
> -    public static void setClientVariable(JMeterVariables var) {
> +    public static void setClientVariables(JMeterVariables var) {
>          variables = var;
> -    }
> +    }
>
>      public static class ThreadCounts {
>
> @@ -197,5 +197,4 @@ public final class JMeterContextService
>              finishedThreads = finished;
>          }
>      }
> -
>  }
>
>

Mime
View raw message