[FLINK-2354] [runtime] Remove state changing futures in JobManager

Internal actor states must only be modified within the actor thread.
This avoids all the well-known issues coming with concurrency.

Fix RemoveCachedJob by introducing RemoveJob

Fix JobManagerITCase

Add removeJob which maintains the job in the SubmittedJobGraphStore

Make revokeLeadership not remove the jobs from the state backend

Fix shading problem with curator by hiding CuratorFramework in ChaosMonkeyITCase

flink-runtime/pom.xml

@@ -182,7 +182,7 @@ under the License.
 
  <dependency>
  <groupId>org.apache.flink</groupId>
- <artifactId>flink-shaded-curator</artifactId>
+ <artifactId>flink-shaded-curator-recipes</artifactId>
  <version>${project.version}</version>
  </dependency>
 
@@ -417,7 +417,7 @@ under the License.
  <configuration>
  <artifactSet>
  <includes combine.children="append">
- <include>org.apache.flink:flink-shaded-curator</include>
+ <include>org.apache.flink:flink-shaded-curator-recipes</include>
  </includes>
  </artifactSet>
  <relocations combine.children="append">