[NIFI-450] FileSystemRepository's subtasks should catch Throwable

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.1.0
• Component/s: Core Framework
• Assignee: Unassigned
• Reporter: Mark Payne
• Created: 19/Mar/15 19:58
• Updated: 26/Mar/15 13:02
• Resolved: 23/Mar/15 19:59

Description

The DestroyExpiredArchiveClaims and BinDestructableClaims classes should wrap their implementations of the run() method with a try

{...} catch (Throwable t) {...}

If any Throwable escapes right now, it causes some pretty bad results, so we shouldn't let this happen. As a result, it could leave behind some files in the repo that won't be cleaned up until restart if a Throwable is thrown, but that's better than letting the thread die.