Summary: | Make CSV Data Set Config use specified encoding | ||
---|---|---|---|
Product: | JMeter - Now in Github | Reporter: | Alf Hogemark <alf> |
Component: | HTTP | Assignee: | JMeter issues mailing list <issues> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | ||
Priority: | P2 | ||
Version: | 2.2 | ||
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | All | ||
Attachments: | Suggested patch to add encoding to CSV data set |
Description
Alf Hogemark
2007-02-26 05:39:39 UTC
Created attachment 19637 [details]
Suggested patch to add encoding to CSV data set
This is a patch agaist SVN
svn.apache.org/repos/asf/jakarta/jmeter/branches/rel-2-2 as of today.
Comments / suggestions to the patch are welcomed.
Thanks for the contribution. Looks mostly OK. However, it changes the public API of the FileServer class, as the reserveFile method now has an extra parameter. This could cause 3rd party applications to fail. Easy enough to resolve by overloading the method. Also, the encoding could perhaps be added to the FileEntry class instead of using a separate HashMap. I'll see about adding this - no need to recreate the patch at present. This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/1869 |