[SLING-5422] Sling Event - HistoryCleanUpTask removes Jobs younger than configured

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Event 4.0.0
• Fix Version/s: Event 4.0.2
• Component/s: Extensions
• Assignee: Carsten Ziegeler
• Reporter: Christoph Nagel
• Created: 11/Jan/16 14:51
• Updated: 20/Sep/17 12:39
• Resolved: 29/Jan/16 07:37

Description

The HistoryCleanUpTask has to remove all Jobs stored in the JCR older than the given parameter age (a number of minutes).
The current implementation traverses the /var/eventing/jobs tree where parts of the path define the date of a job. Problem: the comparison of the job- and removeDate is wrong. First it compares the years, than months, weeks and days but this doesn't work.

I've forked the project on github and wrote a test and fix. Further I added a test dependency for sling-mock in the pom.xml: Fixes HistoryCleanUpTask date comparison.

Please have a look and if you like I can create a pull request on GitHub.

Issue Links

• is cloned by: SLING-5467 Sling Event - CleanUpTask wrongly marks folders