You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jakarta.apache.org by bu...@apache.org on 2010/10/13 17:13:05 UTC

DO NOT REPLY [Bug 50088] New: getAvgPageBytes in SamplingStatCalculator does not what it says

https://issues.apache.org/bugzilla/show_bug.cgi?id=50088

           Summary: getAvgPageBytes in SamplingStatCalculator does not
                    what it says
           Product: JMeter
           Version: 2.4
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Main
        AssignedTo: notifications@jakarta.apache.org
        ReportedBy: jens_0@hotmail.com


getAvgPageBytes in StatCalculator is correct, but in SamplingStatCalculator, it
divides through the duration of the sample (and not, as the comment is saying,
divided by the number ob samples), resulting in about the same value than
geKBPerSecond.

Change this.getElapsed() / 1000 to this.getCount().

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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

DO NOT REPLY [Bug 50088] getAvgPageBytes in SamplingStatCalculator does not what it says

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=50088

Sebb <se...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #1 from Sebb <se...@apache.org> 2010-10-21 09:26:03 EDT ---
Oops! Well spotted. 

Originally there was a method called getPageSize()
This is now renamed as getBytesPerSecond() (because that is what it did).

The intention was to add getAvgPageBytes() to return what the Javadoc says, but
it seems the code copied from getPageSize() code was not amended.

URL: http://svn.apache.org/viewvc?rev=1025989&view=rev
Log:
Bug 50088 - fix getAvgPageBytes in SamplingStatCalculator so it returns what it
should

Modified:
  
jakarta/jmeter/trunk/src/core/org/apache/jmeter/visualizers/SamplingStatCalculator.java
   jakarta/jmeter/trunk/xdocs/changes.xml

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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