Bug 58681 - GraphiteBackendListener : Don't send data if no sampling occured
Summary: GraphiteBackendListener : Don't send data if no sampling occured
Status: RESOLVED FIXED
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: 2.13
Hardware: All All
: P2 normal (vote)
Target Milestone: ---
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-02 20:45 UTC by Philippe Mouawad
Modified: 2015-12-02 20:49 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-02 20:45:16 UTC
Related to:
https://github.com/apache/jmeter/pull/26

Comment:
"The only question I have is that we are recording hits for samplers as well (which is always zero), which does not quite make sense to me. If this metric is present for samplers, then it might mislead users in believing that it should contain valid data and might lead to reported bugs. Would you mind elaborating why you chose this implementation? "
Comment 1 Philippe Mouawad 2015-12-02 20:49:33 UTC
Author: pmouawad
Date: Wed Dec  2 20:48:51 2015
New Revision: 1717681

URL: http://svn.apache.org/viewvc?rev=1717681&view=rev
Log:
Bug 58681 - GraphiteBackendListener : Don't send data if no sampling occured
#resolve #26
Remove useless test of metric.getTotal()
Bugzilla Id: 58681

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/GraphiteBackendListenerClient.java
    jmeter/trunk/xdocs/changes.xml