[DRILL-2575] FragmentExecutor.cancel() blasts through state transitions regardless of current state

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.7.0
• Fix Version/s: 0.8.0
• Component/s: Execution - Flow
• Assignee: Jacques Nadeau
• Reporter: Chris Westin
• Created: 26/Mar/15 00:56
• Updated: 27/Mar/15 16:23
• Resolved: 27/Mar/15 16:23

Description

FragmentExecutor.cancel() unconditionally sets its state to cancelled. In cases where the fragment has already failed or completed successfully, this causes problems. A cancel() in those cases (or when they're in the middle of being finalized) shouldn't do anything.