Bug 58776 - Summariser should display a more readable duration
Summary: Summariser should display a more readable duration
Status: RESOLVED FIXED
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: 2.13
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-28 21:28 UTC by Philippe Mouawad
Modified: 2015-12-29 16:09 UTC (History)
1 user (show)



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philippe Mouawad 2015-12-28 21:28:34 UTC
Currently summariser shows the duration in seconds which is not very readable. 
I always find myself computing the time in hour, minutes seconds.

I propose to change this value to: HH:mm:ss 

replacing:

if (elapsedSec > 100       // No point displaying decimals (less than 1% error)
         || (elapsed - elapsedSec * 1000) < 50 // decimal would be zero
         ) {
            sb.append(longToSb(tmp, elapsedSec, 5));
        } else {
            double elapsedSecf = elapsed / 1000.0d; // fractional seconds
            sb.append(doubleToSb(dfDouble, tmp, elapsedSecf, 5, 1)); // This will round
        }
sb.append("s = ");

by :

sb.append(String.format("%d:%02d:%02d", elapsedSec / 3600, (elapsedSec % 3600) / 60, (elapsedSec % 60)));
sb.append(" = ");
Comment 1 Philippe Mouawad 2015-12-29 16:06:09 UTC
Author: pmouawad
Date: Tue Dec 29 16:05:24 2015
New Revision: 1722174

URL: http://svn.apache.org/viewvc?rev=1722174&view=rev
Log:
Bug 58776 - Summariser should display a more readable duration
Bugzilla Id: 58776

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/reporters/Summariser.java
    jmeter/trunk/xdocs/changes.xml
    jmeter/trunk/xdocs/usermanual/component_reference.xml
Comment 2 Philippe Mouawad 2015-12-29 16:09:50 UTC
Author: pmouawad
Date: Tue Dec 29 16:08:14 2015
New Revision: 1722175

URL: http://svn.apache.org/viewvc?rev=1722175&view=rev
Log:
Fix wrong change commited for Bug 58776
Bugzilla Id: 58776

Modified:
    jmeter/trunk/xdocs/changes.xml