You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by se...@apache.org on 2016/02/13 18:13:36 UTC

svn commit: r1730253 - /jmeter/trunk/src/core/org/apache/jmeter/report/core/CsvSampleReader.java

Author: sebb
Date: Sat Feb 13 17:13:36 2016
New Revision: 1730253

URL: http://svn.apache.org/viewvc?rev=1730253&view=rev
Log:
Report/Dashboard: Improve error reporting
Throw exception if mismatch between expected number of columns and columns in csv file (don't allow missing columns)
Bugzilla Id: 58987

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/report/core/CsvSampleReader.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/report/core/CsvSampleReader.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/core/CsvSampleReader.java?rev=1730253&r1=1730252&r2=1730253&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/core/CsvSampleReader.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/core/CsvSampleReader.java Sat Feb 13 17:13:36 2016
@@ -162,7 +162,7 @@ public class CsvSampleReader implements
             data = CSVSaveService.csvReadFile(reader, separator);
             Sample sample = null;
             if (data.length > 0) {
-                if (data.length < columnCount) {
+                if (data.length != columnCount) {
                     throw new SampleException("Mismatch between expected number of columns:"+columnCount+" and columns in CSV file:"+data.length+
                             ", check your jmeter.save.saveservice.* configuration");
                 }