Bug 58164

Summary: Check if file already exists on ResultCollector listener before starting the loadtest
Product: JMeter Reporter: mchassagneux
Component: MainAssignee: JMeter issues mailing list <issues>
Status: RESOLVED FIXED    
Severity: enhancement CC: p.mouawad
Priority: P2 Keywords: PatchAvailable
Version: 2.13   
Target Milestone: ---   
Hardware: All   
OS: All   
Attachments: Check if file exist on ResultCollector listener before start the loadtest
Nex patch, I forget one file

Description mchassagneux 2015-07-21 13:28:53 UTC
Created attachment 32922 [details]
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.
Comment 1 mchassagneux 2015-07-21 14:21:23 UTC
Created attachment 32924 [details]
Nex patch, I forget one file
Comment 2 Philippe Mouawad 2015-12-20 20:45:34 UTC
If this behaviour is useful, it should also be implemented for Non GUI
Comment 3 Sebb 2016-01-06 00:16:04 UTC
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.
Comment 4 Philippe Mouawad 2017-02-26 15:00:33 UTC
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
Comment 5 Philippe Mouawad 2017-02-26 15:01:50 UTC
Related to PR:
https://github.com/apache/jmeter/pull/247