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 2015/07/21 15:28:53 UTC

[Bug 58164] New: Check if file exist on ResultCollector listener before start the loadtest

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

            Bug ID: 58164
           Summary: Check if file exist on ResultCollector listener before
                    start the loadtest
           Product: JMeter
           Version: 2.13
          Hardware: PC
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: maxime.chassagneux@gmail.com

Created attachment 32922
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=32922&action=edit
Check if file exist on ResultCollector listener before start the loadtest

I made this modification to check and inform the user if a file already exist
on all ResultCollector listener before lauching the loadtest. 

May be, we could add an option to delete automaticaly it ?

Regard.

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

[Bug 58164] Check if file exist on ResultCollector listener before start the loadtest

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO
                 OS|                            |All
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om

--- Comment #2 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
If this behaviour is useful, it should also be implemented for Non GUI

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

[Bug 58164] Check if file already exists on ResultCollector listener before starting the loadtest

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Check if file exist on      |Check if file already
                   |ResultCollector listener    |exists on ResultCollector
                   |before start the loadtest   |listener before starting
                   |                            |the loadtest

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

[Bug 58164] Check if file already exists on ResultCollector listener before starting the loadtest

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

--- Comment #5 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Related to PR:
https://github.com/apache/jmeter/pull/247

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

[Bug 58164] Check if file exist on ResultCollector listener before start the loadtest

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

--- Comment #1 from maxime.chassagneux@gmail.com ---
Created attachment 32924
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=32924&action=edit
Nex patch, I forget one file

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

[Bug 58164] Check if file already exists on ResultCollector listener before starting the loadtest

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |PatchAvailable
           Hardware|PC                          |All

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

[Bug 58164] Check if file already exists on ResultCollector listener before starting the loadtest

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Author: mchassagneux
Date: Thu Feb 16 21:23:03 2017
New Revision: 1783297

URL: http://svn.apache.org/viewvc?rev=1783297&view=rev
Log:
Checks for listener output file existence
Bugzilla Id: 58164

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/JMeter.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/AbstractAction.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/RemoteStart.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/Start.java
    jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties
    jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties
    jmeter/trunk/xdocs/changes.xml

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

[Bug 58164] Check if file exist on ResultCollector listener before start the loadtest

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

--- Comment #3 from Sebb <se...@apache.org> ---
I think this behaviour needs to be configurable if it is added.

I often run tests where appending the output is OK, and having a pop-up would
be a big nuisance.

Also pop-ups won't work for non-GUI.

So I would rather see an option which if selected, automatically deletes any
existing file.

And/or could have an option to automatically stop the test if the file exists.

Note also that the same file may potentially be used by multiple Listeners.
In that case, the file must only be deleted by the first instance.

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