[FLINK-5082] Pull ExecutionService lifecycle management out of the JobManager

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.1.3, 1.2.0
• Fix Version/s: 1.1.4, 1.2.0
• Component/s: Runtime / Coordination
• Assignee: Till Rohrmann
• Reporter: Till Rohrmann
• Created: 16/Nov/16 17:33
• Updated: 28/Feb/19 14:01
• Resolved: 22/Nov/16 22:12

Description

The JobManager receives an ExecutorService to run its futures as a constructor parameter. Even though the ExecutorService comes from outside, the JobManager shuts the executor service down if the JobManager terminates. This is clearly a sub-optimal behaviour leading also to RejectedExecutionExceptions.

I propose to move the ExecutorService lifecycle management out of the JobManager.

Issue Links

• links to: GitHub Pull Request #2820