You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2017/01/26 20:50:28 UTC
svn commit: r1780476 - in
/jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend:
graphite/PickleGraphiteMetricsSender.java
graphite/TextGraphiteMetricsSender.java influxdb/HttpMetricsSender.java
Author: pmouawad
Date: Thu Jan 26 20:50:28 2017
New Revision: 1780476
URL: http://svn.apache.org/viewvc?rev=1780476&view=rev
Log:
Fix sonar code smells
Modified:
jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/PickleGraphiteMetricsSender.java
jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/TextGraphiteMetricsSender.java
jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/influxdb/HttpMetricsSender.java
Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/PickleGraphiteMetricsSender.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/PickleGraphiteMetricsSender.java?rev=1780476&r1=1780475&r2=1780476&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/PickleGraphiteMetricsSender.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/PickleGraphiteMetricsSender.java Thu Jan 26 20:50:28 2017
@@ -103,7 +103,7 @@ class PickleGraphiteMetricsSender extend
*/
@Override
public void writeAndSendMetrics() {
- List<MetricTuple> tempMetrics = null;
+ List<MetricTuple> tempMetrics;
synchronized (lock) {
if(metrics.isEmpty()) {
return;
Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/TextGraphiteMetricsSender.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/TextGraphiteMetricsSender.java?rev=1780476&r1=1780475&r2=1780476&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/TextGraphiteMetricsSender.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/graphite/TextGraphiteMetricsSender.java Thu Jan 26 20:50:28 2017
@@ -86,7 +86,7 @@ class TextGraphiteMetricsSender extends
*/
@Override
public void writeAndSendMetrics() {
- List<MetricTuple> tempMetrics = null;
+ List<MetricTuple> tempMetrics;
synchronized (lock) {
if(metrics.isEmpty()) {
return;
Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/influxdb/HttpMetricsSender.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/influxdb/HttpMetricsSender.java?rev=1780476&r1=1780475&r2=1780476&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/influxdb/HttpMetricsSender.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/backend/influxdb/HttpMetricsSender.java Thu Jan 26 20:50:28 2017
@@ -143,7 +143,7 @@ class HttpMetricsSender extends Abstract
*/
@Override
public void writeAndSendMetrics() {
- List<MetricTuple> tempMetrics = null;
+ List<MetricTuple> tempMetrics;
synchronized (lock) {
if(metrics.isEmpty()) {
return;