You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2012/08/28 22:20:29 UTC
svn commit: r1378304 - in
/jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test:
JavaTest.java SleepTest.java
Author: sebb
Date: Tue Aug 28 20:20:29 2012
New Revision: 1378304
URL: http://svn.apache.org/viewvc?rev=1378304&view=rev
Log:
Enhance JavaSampler handling of JavaSamplerClient cleanup to use less memory
Remove unnecessary implementations of tearDownTest - they just waste memory
Bugzilla Id: 53782
Modified:
jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/JavaTest.java
jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/SleepTest.java
Modified: jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/JavaTest.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/JavaTest.java?rev=1378304&r1=1378303&r2=1378304&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/JavaTest.java (original)
+++ jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/JavaTest.java Tue Aug 28 20:20:29 2012
@@ -321,22 +321,6 @@ public class JavaTest extends AbstractJa
}
/**
- * Do any clean-up required by this test. In this case no clean-up is
- * necessary, but some messages are logged for debugging purposes.
- *
- * @param context
- * the context to run with. This provides access to
- * initialization parameters.
- */
- @Override
- public void teardownTest(JavaSamplerContext context) {
- if (LOG.isDebugEnabled()) {
- LOG.debug(whoAmI() + "\tteardownTest()");
- listParameters(context);
- }
- }
-
- /**
* Dump a list of the parameters in this context to the debug log.
* Should only be called if debug is enabled.
*
Modified: jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/SleepTest.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/SleepTest.java?rev=1378304&r1=1378303&r2=1378304&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/SleepTest.java (original)
+++ jmeter/trunk/src/protocol/java/org/apache/jmeter/protocol/java/test/SleepTest.java Tue Aug 28 20:20:29 2012
@@ -166,22 +166,6 @@ public class SleepTest extends AbstractJ
}
/**
- * Do any clean-up required by this test. In this case no clean-up is
- * necessary, but some messages are logged for debugging purposes.
- *
- * @param context
- * the context to run with. This provides access to
- * initialization parameters.
- */
- @Override
- public void teardownTest(JavaSamplerContext context) {
- if (LOG.isDebugEnabled()) {
- LOG.debug(whoAmI() + "\tteardownTest()");
- listParameters(context);
- }
- }
-
- /**
* Provide a list of parameters which this test supports. Any parameter
* names and associated values returned by this method will appear in the
* GUI by default so the user doesn't have to remember the exact names. The