Issue Details (XML | Word | Printable)

Key: MPCLOVER-48
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Critical Critical
Assignee: Vincent Massol
Reporter: Brendan Humphreys
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Maven 1.x Clover Plugin

allow override of flushpolicy, flushinterval in <clover-setup>, change default values

Created: 28/Sep/05 09:25 PM   Updated: 29/Sep/05 04:32 AM   Resolved: 29/Sep/05 04:32 AM
Return to search
Component/s: None
Affects Version/s: 1.10
Fix Version/s: 1.11

Time Tracking:
Not Specified


 Description  « Hide

plugin.jelly hardwires Clover to use flushpolicy="interval" flushinterval="500". For large projects this can lead to a severe runtime performance impact because Clover tries to flush the coverage data to disk every 500msecs. I suggest exposing the flushpolicy and flushinterval as plugin properties, and also changing the default values:

plugin.properties:
maven.clover.flushpolicy=threaded
maven.clover.flushinterval=5000

plugin.jelly:
<ant:clover-setup
initstring="${maven.clover.database}"
flushpolicy="${maven.clover.flushpolicy}"
flushinterval="${maven.clover.flushinterval}">



Vincent Massol added a comment - 29/Sep/05 04:32 AM

Applied.Thanks Brendan.

Note: I've had a failure the first time I ran the clover plugin tests. I wasn't able to reproduce it later on. I hope this is not related to any time-issue while flushing coverage data to the database due to the change of policy.