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 2016/08/02 19:32:20 UTC
svn commit: r1754997 -
/jmeter/trunk/src/components/org/apache/jmeter/config/CSVDataSet.java
Author: pmouawad
Date: Tue Aug 2 19:32:20 2016
New Revision: 1754997
URL: http://svn.apache.org/viewvc?rev=1754997&view=rev
Log:
Avoid NPE later when using vars. Throw directly an IAE with informations about issue
Modified:
jmeter/trunk/src/components/org/apache/jmeter/config/CSVDataSet.java
Modified: jmeter/trunk/src/components/org/apache/jmeter/config/CSVDataSet.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/config/CSVDataSet.java?rev=1754997&r1=1754996&r2=1754997&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/config/CSVDataSet.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/config/CSVDataSet.java Tue Aug 2 19:32:20 2016
@@ -180,7 +180,7 @@ public class CSVDataSet extends ConfigTe
vars = CSVSaveService.csvSplitString(header, delim.charAt(0));
firstLineIsNames = true;
} catch (IOException e) {
- log.warn("Could not split CSV header line",e);
+ throw new IllegalArgumentException("Could not split CSV header line from file:" + _fileName,e);
}
} else {
server.reserveFile(_fileName, getFileEncoding(), alias);