You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@jmeter.apache.org by bu...@apache.org on 2016/02/12 21:18:52 UTC

[Bug 59000] New: Prevent mixed CSV columns

https://bz.apache.org/bugzilla/show_bug.cgi?id=59000

            Bug ID: 59000
           Summary: Prevent mixed CSV columns
           Product: JMeter
           Version: 2.13
          Hardware: PC
                OS: Mac OS X 10.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: sebb@apache.org

JMeter by default appends data to an existing response data file.

If the columm list changes this causes problems for CSV files.

When appending to an existing CSV file, the code should check if the settings
are compatible.

If not, then some action should be taken, e.g. abandon the test or rename the
output file so the existing file is not compromised.

If the existing file has a header line, then a simple comparison of the text is
sufficient to determine if the column settings are equal.

For files without headers, the code can check if the column count is the same.
It can potentially also check whether the expected columns are numeric.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59000] Prevent mixed CSV columns

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59000

--- Comment #2 from Sebb <se...@apache.org> ---
(In reply to Philippe Mouawad from comment #1)
> Isn't it easier and better to just not append ?
> Either fail if target file exists or rename it ?

JMeter has always allowed output to be appended, and that works fine if the
settings are not changed. Indeed it can be quite useful.

Disallowing all appends would break customer scripts that rely on the
behaviour; we cannot arbitrarily change how JMeter works.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59000] Prevent mixed CSV columns

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59000

Sebb <se...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |NEW

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59000] Prevent mixed CSV columns

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59000

Sebb <se...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|Mac OS X 10.1               |All
           Hardware|PC                          |All

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59000] Prevent mixed CSV columns

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59000

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om
           Severity|normal                      |enhancement

--- Comment #1 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Isn't it easier and better to just not append ?
Either fail if target file exists or rename it ?

-- 
You are receiving this mail because:
You are the assignee for the bug.