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

HTML Report Generator: Add Graph Total Transactions per Second #4779

Closed
asfimport opened this issue May 10, 2018 · 9 comments
Closed

HTML Report Generator: Add Graph Total Transactions per Second #4779

asfimport opened this issue May 10, 2018 · 9 comments

Comments

@asfimport
Copy link
Collaborator

laks.martha (Bug 62367):
Hello, We are looking for Total Transactions per second graph in HTML Report. Currently in 4.0 code it is generating transactions per second graph. So we need similar graph aggregation by Time and Transaction status.

OS: All

@asfimport
Copy link
Collaborator Author

laks.martha (migrated from Bugzilla):
Created attachment jmeter-totaltps-graph-enhancement.zip: Jmeter Total TPS Graph generator enhancement.

@asfimport
Copy link
Collaborator Author

laks.martha (migrated from Bugzilla):
This attachement contains source code and property changes for Total TPS graph enhancement.

Add new class file
core\org\jmeter\report\processor\graph\impl\TotalTPSGraphConsumer.java

Add new properties in reportgenerator.properties
jmeter.reportgenerator.graph.totalTPS.*

Added few lines of code in below files to display new graph
report-template\content\pages\Throughput.html.fmkr
report-template\content\js\graph.js.fmkr

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
Thanks for contribution.
Could you attach a screenshot of result please ?

Thanks

@asfimport
Copy link
Collaborator Author

laks.martha (migrated from Bugzilla):
Created attachment TotalTPS.png: Screenshot of Total TPS graph after applying changes.

Screenshot of Total TPS graph after applying changes.

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
Author: pmouawad
Date: Fri May 11 20:00:39 2018
New Revision: 1831444

URL: http://svn.apache.org/viewvc?rev=1831444&view=rev
Log:
#4779 - HTML Report Generator: Add Graph Total Transactions per Second
Mainly contributed by laks.martha
#4779

Added:
jmeter/trunk/src/core/org/apache/jmeter/report/processor/graph/impl/TotalTPSGraphConsumer.java (with props)
Modified:
jmeter/trunk/bin/report-template/content/js/graph.js.fmkr
jmeter/trunk/bin/report-template/content/pages/Throughput.html.fmkr
jmeter/trunk/bin/reportgenerator.properties
jmeter/trunk/xdocs/changes.xml

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
Hello,
Thanks for this contribution !

I merged it with slight modifications:

  • Fixed the colors (red for errors, green for ok)
  • Made the granularity variable

Can you test and check it's ok ?
Thanks

@asfimport
Copy link
Collaborator Author

laks.martha (migrated from Bugzilla):
Thanks i have downloaded config files with color changes and tested. Its working as expected.

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
Graph appears blank as of nightly build.
Reopening

@asfimport
Copy link
Collaborator Author

@pmouawad (migrated from Bugzilla):
Author: pmouawad
Date: Sun Jul 15 13:58:17 2018
New Revision: 1835977

URL: http://svn.apache.org/viewvc?rev=1835977&view=rev
Log:
#4779 - HTML Report Generator: Add Graph Total Transactions per Second
Fix empty graph when a jmeter.reportgenerator.exporter.html.series_filter is defined
#4779

Modified:
jmeter/trunk/bin/report-template/content/js/dashboard-commons.js.fmkr
jmeter/trunk/bin/report-template/content/js/graph.js.fmkr

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