Bug 58164 - Check if file already exists on ResultCollector listener before starting the loadtest
Summary: Check if file already exists on ResultCollector listener before starting the ...
Status: RESOLVED FIXED
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: 2.13
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: JMeter issues mailing list
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2015-07-21 13:28 UTC by maxime.chassagneux
Modified: 2017-02-26 15:02 UTC (History)
1 user (show)



Attachments
Check if file exist on ResultCollector listener before start the loadtest (1.14 KB, patch)
2015-07-21 13:28 UTC, maxime.chassagneux
Details | Diff
Nex patch, I forget one file (2.46 KB, patch)
2015-07-21 14:21 UTC, maxime.chassagneux
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description maxime.chassagneux 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 maxime.chassagneux 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