Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/4697#discussion_r141111201
--- 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.
--- End diff --
it would be more consistent with the javadoc of other method to say
"Cancels"---
