| Summary: | Summariser should display a more readable duration | ||
|---|---|---|---|
| Product: | JMeter - Now in Github | Reporter: | Philippe Mouawad <p.mouawad> |
| Component: | Main | Assignee: | JMeter issues mailing list <issues> |
| Status: | RESOLVED FIXED | ||
| Severity: | enhancement | CC: | p.mouawad |
| Priority: | P2 | ||
| Version: | 2.13 | ||
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | All | ||
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 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 This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/3745 |
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(" = ");