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 2012/09/27 22:28:32 UTC

svn commit: r1391201 - in /jmeter/trunk: src/components/org/apache/jmeter/visualizers/ src/core/org/apache/jmeter/resources/ src/core/org/apache/jmeter/visualizers/ xdocs/

Author: pmouawad
Date: Thu Sep 27 20:28:31 2012
New Revision: 1391201

URL: http://svn.apache.org/viewvc?rev=1391201&view=rev
Log:
Bug 53941 - Add latency to View Result table listener
Bugzilla Id: 53941

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/TableVisualizer.java
    jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties
    jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties
    jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java
    jmeter/trunk/xdocs/changes.xml

Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/TableVisualizer.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/TableVisualizer.java?rev=1391201&r1=1391200&r2=1391201&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/TableVisualizer.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/TableVisualizer.java Thu Sep 27 20:28:31 2012
@@ -80,7 +80,8 @@ public class TableVisualizer extends Abs
             "sampler_label",                // $NON-NLS-1$
             "table_visualizer_sample_time", // $NON-NLS-1$
             "table_visualizer_status",      // $NON-NLS-1$
-            "table_visualizer_bytes" };     // $NON-NLS-1$
+            "table_visualizer_bytes",       // $NON-NLS-1$
+            "table_visualizer_latency"};    // $NON-NLS-1$
 
     private ObjectTableModel model = null;
 
@@ -131,10 +132,11 @@ public class TableVisualizer extends Abs
                 new Functor("getLabel"), // $NON-NLS-1$
                 new Functor("getElapsed"), // $NON-NLS-1$
                 new SampleSuccessFunctor("isSuccess"), // $NON-NLS-1$
-                new Functor("getBytes") }, // $NON-NLS-1$
-                new Functor[] { null, null, null, null, null, null, null },
+                new Functor("getBytes"), // $NON-NLS-1$
+                new Functor("getLatency") }, // $NON-NLS-1$
+                new Functor[] { null, null, null, null, null, null, null, null },
                 new Class[] {
-                String.class, String.class, String.class, String.class, Long.class, ImageIcon.class, Integer.class });
+                String.class, String.class, String.class, String.class, Long.class, ImageIcon.class, Long.class, Long.class });
         init();
     }
 
@@ -178,7 +180,8 @@ public class TableVisualizer extends Abs
                             res.getSampleLabel(),
                             res.getTime(),
                             res.isSuccessful(),
-                            res.getBytes());
+                            res.getBytes(),
+                            res.getLatency());
                     model.addRow(newS);
                 }
                 updateTextFields(res);

Modified: jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties?rev=1391201&r1=1391200&r2=1391201&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties Thu Sep 27 20:28:31 2012
@@ -994,6 +994,7 @@ system_sampler_stdin=Standard input (std
 system_sampler_stdout=Standard output (stdout):
 system_sampler_stderr=Standard error (stderr):
 table_visualizer_bytes=Bytes
+table_visualizer_latency=Latency
 table_visualizer_sample_num=Sample #
 table_visualizer_sample_time=Sample Time(ms)
 table_visualizer_start_time=Start Time

Modified: jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties?rev=1391201&r1=1391200&r2=1391201&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties Thu Sep 27 20:28:31 2012
@@ -988,6 +988,7 @@ system_sampler_stdin=Entr\u00E9e standar
 system_sampler_stdout=Sortie standard (stdout) \:
 system_sampler_stderr=Erreur standard (stderr) \:
 table_visualizer_bytes=Octets
+table_visualizer_latency=Latence
 table_visualizer_sample_num=Echantillon \#
 table_visualizer_sample_time=Temps (ms)
 table_visualizer_start_time=Heure d\u00E9but

Modified: jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java?rev=1391201&r1=1391200&r2=1391201&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java Thu Sep 27 20:28:31 2012
@@ -44,17 +44,19 @@ public class TableSample implements Seri
 
     private final long bytes;
 
+    private final long latency;
+
     /**
      * @deprecated for unit test code only
      */
     @Deprecated
     public TableSample() {
-        this(0, 1, 0, "", "", 0, true, 0);
+        this(0, 1, 0, "", "", 0, true, 0, 0);
     }
 
     public TableSample(long totalSamples, int sampleCount, long startTime, String threadName,
             String label,
-            long elapsed, boolean success, long bytes) {
+            long elapsed, boolean success, long bytes, long latency) {
         this.totalSamples = totalSamples;
         this.sampleCount = sampleCount;
         this.startTime = startTime;
@@ -63,6 +65,7 @@ public class TableSample implements Seri
         this.elapsed = elapsed/sampleCount;
         this.success = success;
         this.bytes = bytes/sampleCount;
+        this.latency = latency;
     }
 
     // The following getters may appear not to be used - however they are invoked via the Functor class
@@ -129,4 +132,11 @@ public class TableSample implements Seri
         return (int)(totalSamples ^ (totalSamples >>> 32));
     }
 
+    /**
+     * @return the latency
+     */
+    public long getLatency() {
+        return latency;
+    }
+
 }

Modified: jmeter/trunk/xdocs/changes.xml
URL: http://svn.apache.org/viewvc/jmeter/trunk/xdocs/changes.xml?rev=1391201&r1=1391200&r2=1391201&view=diff
==============================================================================
--- jmeter/trunk/xdocs/changes.xml (original)
+++ jmeter/trunk/xdocs/changes.xml Thu Sep 27 20:28:31 2012
@@ -191,6 +191,7 @@ The original behaviour can be restored b
 <li><bugzilla>53749</bugzilla> - TestListener interface could perhaps be split up. 
 This should reduce per-thread memory requirements and processing, 
 as only test elements that actually use testIterationStart functionality now need to be handled.</li>
+<li><bugzilla>53941</bugzilla> - Add latency to View Result table listener</li>
 </ul>
 
 <h3>Timers, Assertions, Config, Pre- &amp; Post-Processors</h3>