Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/4697#discussion_r141111338
--- 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 --
what is returned if the cancellation was not successful?
---
