You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jakarta.apache.org by se...@apache.org on 2011/01/28 17:04:51 UTC

svn commit: r1064752 - in /jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers: BatchSampleSender.java DataStrippingSampleSender.java HoldSampleSender.java StandardSampleSender.java StatisticalSampleSender.java

Author: sebb
Date: Fri Jan 28 16:04:51 2011
New Revision: 1064752

URL: http://svn.apache.org/viewvc?rev=1064752&view=rev
Log:
Clean up logging - drop per-sample to reduce load

Modified:
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/BatchSampleSender.java
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/DataStrippingSampleSender.java
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/HoldSampleSender.java
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StandardSampleSender.java
    jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleSender.java

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/BatchSampleSender.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/BatchSampleSender.java?rev=1064752&r1=1064751&r2=1064752&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/BatchSampleSender.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/BatchSampleSender.java Fri Jan 28 16:04:51 2011
@@ -52,6 +52,12 @@ public class BatchSampleSender implement
 
     private long batchSendTime = -1;
 
+    static {
+        log.info("Using batching for this run."
+                + " Thresholds: num=" + NUM_SAMPLES_THRESHOLD
+                + ", time=" + TIME_THRESHOLD_MS);        
+    }
+
     /**
      * @deprecated only for use by test code
      */
@@ -68,9 +74,6 @@ public class BatchSampleSender implement
      */
     protected BatchSampleSender(RemoteSampleListener listener) {
         this.listener = listener;
-        log.info("Using batching for this run."
-                + " Thresholds: num=" + NUM_SAMPLES_THRESHOLD
-                + ", time=" + TIME_THRESHOLD_MS);
     }
 
    /**

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/DataStrippingSampleSender.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/DataStrippingSampleSender.java?rev=1064752&r1=1064751&r2=1064752&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/DataStrippingSampleSender.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/DataStrippingSampleSender.java Fri Jan 28 16:04:51 2011
@@ -38,6 +38,9 @@ public class DataStrippingSampleSender i
     private final RemoteSampleListener listener;
     private final SampleSender decoratedSender;
 
+    static {
+        log.info("Using DataStrippingSampleSender for this run");
+    }
     /**
      * @deprecated only for use by test code
      */

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/HoldSampleSender.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/HoldSampleSender.java?rev=1064752&r1=1064751&r2=1064752&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/HoldSampleSender.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/HoldSampleSender.java Fri Jan 28 16:04:51 2011
@@ -40,6 +40,10 @@ public class HoldSampleSender implements
 
     private final RemoteSampleListener listener;
 
+    static {
+        log.info("Using Sample store for this test run");        
+    }
+
     /**
      * @deprecated only for use by test code
      */
@@ -50,13 +54,12 @@ public class HoldSampleSender implements
     }
 
     HoldSampleSender(RemoteSampleListener listener) {
-        log.info("Using Sample store for this test run");
         this.listener = listener;
     }
 
     /** {@inheritDoc} */
     public void testEnded() {
-        log.info("Test ended()");
+        log.debug("Test ended()");
         try {
             synchronized (sampleStore) {
                 Iterator<SampleEvent> i = sampleStore.iterator();
@@ -81,7 +84,7 @@ public class HoldSampleSender implements
 
     /** {@inheritDoc} */
     public void testEnded(String host) {
-        log.info("Test Ended on " + host); // should this be debug?
+        log.debug("Test Ended on " + host);
         try {
             Iterator<SampleEvent> i = sampleStore.iterator();
             while (i.hasNext()) {
@@ -104,7 +107,6 @@ public class HoldSampleSender implements
 
     /** {@inheritDoc} */
     public void sampleOccurred(SampleEvent e) {
-        log.debug("Sample occurred");
         synchronized (sampleStore) {
             sampleStore.add(e);
         }

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StandardSampleSender.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StandardSampleSender.java?rev=1064752&r1=1064751&r2=1064752&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StandardSampleSender.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StandardSampleSender.java Fri Jan 28 16:04:51 2011
@@ -36,6 +36,10 @@ public class StandardSampleSender implem
 
     private RemoteSampleListener listener;
 
+    static {
+        log.info("Using Standard Remote Sampler for this test run");        
+    }
+
     /**
      * @deprecated only for use by test code
      */
@@ -45,7 +49,6 @@ public class StandardSampleSender implem
     }
 
     StandardSampleSender(RemoteSampleListener listener) {
-        log.info("Using Standard Remote Sampler for this test run");
         this.listener = listener;
     }
 
@@ -69,7 +72,6 @@ public class StandardSampleSender implem
     }
 
     public void sampleOccurred(SampleEvent e) {
-        log.debug("Sample occurred");
         try {
             listener.sampleOccurred(e);
         } catch (RemoteException err) {

Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleSender.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleSender.java?rev=1064752&r1=1064751&r2=1064752&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleSender.java (original)
+++ jakarta/jmeter/trunk/src/core/org/apache/jmeter/samplers/StatisticalSampleSender.java Fri Jan 28 16:04:51 2011
@@ -62,6 +62,12 @@ public class StatisticalSampleSender imp
 
     private long batchSendTime = -1;
 
+    static {
+        log.info("Using statistical sampling for this run." + " Thresholds: num="
+                + NUM_SAMPLES_THRESHOLD + ", time=" + TIME_THRESHOLD_MS
+                + ". Key uses ThreadName: " + KEY_ON_THREADNAME);        
+    }
+
     /**
      * @deprecated only for use by test code
      */
@@ -71,7 +77,6 @@ public class StatisticalSampleSender imp
         listener = null;
     }
 
-
     /**
      * Constructor
      *
@@ -79,9 +84,6 @@ public class StatisticalSampleSender imp
      */
     StatisticalSampleSender(RemoteSampleListener listener) {
         this.listener = listener;
-        log.info("Using statistical sampling for this run." + " Thresholds: num="
-                + NUM_SAMPLES_THRESHOLD + ", time=" + TIME_THRESHOLD_MS
-                + ". Key uses ThreadName: " + KEY_ON_THREADNAME);
     }
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@jakarta.apache.org
For additional commands, e-mail: notifications-help@jakarta.apache.org