This is an automated email from the ASF dual-hosted git repository. fschumacher pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/jmeter.git
commit 8d53bb9491c629696b48b233605b3d70036f611a Author: Felix Schumacher <[email protected]> AuthorDate: Sat Jul 9 14:15:11 2022 +0200 Don't generate same string twice. --- .../org/apache/jmeter/report/core/CsvSampleReader.java | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/src/core/src/main/java/org/apache/jmeter/report/core/CsvSampleReader.java b/src/core/src/main/java/org/apache/jmeter/report/core/CsvSampleReader.java index 97508d58b5..86b134c805 100644 --- a/src/core/src/main/java/org/apache/jmeter/report/core/CsvSampleReader.java +++ b/src/core/src/main/java/org/apache/jmeter/report/core/CsvSampleReader.java @@ -150,18 +150,14 @@ public class CsvSampleReader implements Closeable{ && CSVSaveService.getSampleSaveConfiguration( line, file.getAbsolutePath()) == null) { // Build metadata from default save config - if (log.isWarnEnabled()) { - log.warn( - "File '{}' does not contain the field names header, " - + "ensure the jmeter.save.saveservice.* properties are the same " - + "as when the CSV file was created or the file may be read incorrectly " - + "when generating report", - file.getAbsolutePath()); - } - System.out.println("File '"+file.getAbsolutePath()+"' does not contain the field names header, " + String warnMessage = "File '" + file.getAbsolutePath() + "' does not contain the field names header, " + "ensure the jmeter.save.saveservice.* properties are the same " + "as when the CSV file was created or the file may be read incorrectly " - + "when generating report"); + + "when generating report"; + if (log.isWarnEnabled()) { + log.warn(warnMessage); + } + System.out.println(warnMessage); result = new SampleMetadata( SampleSaveConfiguration.staticConfig());
