[GitHub] flink pull request #5881: [FLINK-9190][yarn] fix YarnResourceManager sometim...

[sihuazhou]

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

    https://github.com/apache/flink/pull/5881#discussion_r183265730
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/YarnResourceManager.java ---
    @@ -265,6 +272,29 @@ protected void initialize() throws 
ResourceManagerException {
                }
        }
     
    +   @Override
    +   public CompletableFuture<RegistrationResponse> registerTaskExecutor(
    --- End diff --
    
    Good point! Have to change.


---