Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Summariser should display a more readable duration #3745

Closed
asfimport opened this issue Dec 28, 2015 · 2 comments
Closed

Summariser should display a more readable duration #3745

asfimport opened this issue Dec 28, 2015 · 2 comments

Comments

@asfimport
Copy link
Collaborator

@pmouawad (Bug 58776):
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(" = ");

OS: All

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
Author: pmouawad
Date: Tue Dec 29 16:05:24 2015
New Revision: 1722174

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

Modified:
jmeter/trunk/src/core/org/apache/jmeter/reporters/Summariser.java
jmeter/trunk/xdocs/changes.xml
jmeter/trunk/xdocs/usermanual/component_reference.xml

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
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 #3745
#3745

Modified:
jmeter/trunk/xdocs/changes.xml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant