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 2011/11/14 20:55:21 UTC
svn commit: r1201853 -
/jmeter/trunk/src/monitor/components/org/apache/jmeter/visualizers/MonitorAccumModel.java
Author: pmouawad
Date: Mon Nov 14 19:55:21 2011
New Revision: 1201853
URL: http://svn.apache.org/viewvc?rev=1201853&view=rev
Log:
Fixed NullPointerException when URL is null (supposed to be a file reload)
Modified:
jmeter/trunk/src/monitor/components/org/apache/jmeter/visualizers/MonitorAccumModel.java
Modified: jmeter/trunk/src/monitor/components/org/apache/jmeter/visualizers/MonitorAccumModel.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/monitor/components/org/apache/jmeter/visualizers/MonitorAccumModel.java?rev=1201853&r1=1201852&r2=1201853&view=diff
==============================================================================
--- jmeter/trunk/src/monitor/components/org/apache/jmeter/visualizers/MonitorAccumModel.java (original)
+++ jmeter/trunk/src/monitor/components/org/apache/jmeter/visualizers/MonitorAccumModel.java Mon Nov 14 19:55:21 2011
@@ -167,9 +167,13 @@ public class MonitorAccumModel implement
MonitorModel mo = new MonitorModel(stat);
this.addSample(mo);
notifyListeners(mo);
- } else {
- noResponse(surl);
- }
+ }
+ // This part of code throws NullPointerException
+ // Don't think Monitor results can be loaded from files
+ // see https://issues.apache.org/bugzilla/show_bug.cgi?id=51810
+// else {
+// noResponse(surl);
+// }
} else if (((HTTPSampleResult) sample).isMonitor()) {
noResponse(surl);
}