Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

result file can not be jtl in V2.7.0,if you want to generate jtl,you must configurate no reports #282

Closed
andyzhou192 opened this issue Aug 3, 2018 · 4 comments

Comments

@andyzhou192
Copy link

andyzhou192 commented Aug 3, 2018

result file can not be jtl in V2.7.0,if you want to generate jtl,you must configurate no reports,
eg:
false jtl

, source code like this :"
protected void setJMeterResultFileFormat() {
if (generateReports || "csv".equalsIgnoreCase(resultsFileFormat)) {
propertiesJMeter.put("jmeter.save.saveservice.output_format", "csv");
resultsOutputIsCSVFormat = true;
} else {
propertiesJMeter.put("jmeter.save.saveservice.output_format", "xml");
resultsOutputIsCSVFormat = false;
}
}

@andyzhou192 andyzhou192 changed the title result file can not be jtl in V2.7.0,if you want to generate jtl,you must configurate no reports,like "<generateReports>false</generateReports> <resultsFileFormat>jtl</resultsFileFormat>", source code like this :"protected void setJMeterResultFileFormat() { if (generateReports || "csv".equalsIgnoreCase(resultsFileFormat)) { propertiesJMeter.put("jmeter.save.saveservice.output_format", "csv"); resultsOutputIsCSVFormat = true; } else { propertiesJMeter.put("jmeter.save.saveservice.output_format", "xml"); resultsOutputIsCSVFormat = false; } }" result file can not be jtl in V2.7.0,if you want to generate jtl,you must configurate no reports Aug 3, 2018
@pmouawad
Copy link
Contributor

pmouawad commented Aug 3, 2018

Hello,
What is the problem here ?
This has been documented as part of ChangeLog:
https://github.com/jmeter-maven-plugin/jmeter-maven-plugin/blob/master/CHANGELOG.md

Regards

@pmouawad pmouawad added this to Low priority in ISSUE TRACKING Aug 3, 2018
@pmouawad pmouawad moved this from Low priority to Needs triage in ISSUE TRACKING Aug 3, 2018
@adbatrockend
Copy link

My comment at #266 (comment) mentioned that the documentation doesn't mention generateReports, so users may be confused into thinking that setting <resultsFileFormat>xml</resultsFileFormat> is the only thing that needs to be done (most user's aren't going to look at the CHANGELOG).

Possibly just updating the documentation is all that's required?

@pmouawad
Copy link
Contributor

I updated wiki let me know if ok for you

ISSUE TRACKING automation moved this from Needs triage to Closed Aug 23, 2018
@andyzhou192
Copy link
Author

OK,thx!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
ISSUE TRACKING
  
Closed
Development

No branches or pull requests

3 participants