One of the FileServer#reserveFile() methods allows the user to specify that a file has a header line, in which case it is returned. [The others don't allow a header] However, when reading a headed file later, the user has to remember to provide the correct parameter to readLine. This is a recipe for confusion. The reserveFile() methods should save the header status of the file entry when it is created. This should then be used by the the readLine() methods, which should not need to be told the header status. However, changing this might possibly break some 3rd party code. So perhaps the safest would be to deprecate the following method: readLine(String filename, boolean recycle, boolean firstLineIsNames) and log a warning (or error?) if the reading mode disagrees with the reservation mode.
Likewise, the recycle setting should be fixed when reserving the file.
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/2907