Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4697#discussion_r141295793
  
    --- 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 --
    
    Hmm, not sure if that is true tbh. We actually have quite a wild mix of how 
we write JavaDocs in `JobMasterGateway`. I will change all of them starting 
with a verb in the 3rd person.


---

Reply via email to