[PIG-3524] Clean up Launcher and MapReduceLauncher after PIG-3419

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.13.0
• Fix Version/s: 0.13.0
• Component/s: impl
• Patch Info: Patch Available
• Assignee: Cheolsoo Park
• Reporter: Cheolsoo Park
• Created: 17/Oct/13 06:33
• Updated: 04/Jun/16 21:30
• Resolved: 19/Oct/13 04:36

Description

This is clean-up that I did for Launcher and MapReduceLauncher as part of PIG-3502. I'd like to merge applicable changes from tez branch to trunk so that they will not diverge too much over time.

The changes include:

1. Rename ExecutionEngineKiller to HangingJobKiller and move it from PigContext to Launcher. This brings back HangingJobKiller to its original place before PIG-3419.
2. Remove kill() method from the ExecutionEngine interface. It was needed only by ExecutionEngineKiller, but after #1, it's no longer needed.
3. Refactor Launcher and MapReduceLauncer classes. Basically, move MR-specific stuff to MapReduceLauncher and common stuff to Launcher.

Issue Links

• breaks: PIG-4921 Kill running jobs on InterruptedException