Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/4697#discussion_r141298335
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/jobmaster/JobMasterGateway.java
---
@@ -55,6 +55,14 @@
public interface JobMasterGateway extends CheckpointCoordinatorGateway,
FencedRpcGateway<JobMasterId> {
/**
+ * Cancel the currently executed job.
+ *
+ * @param timeout of this operation
+ * @return Future acknowledge if the cancellation was successful
--- End diff --
at the moment it cannot fail
---