[GitHub] flink pull request #6087: [FLINK-9410] [yarn] Replace NMClient with NMClient...

[zhangminglei]

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

    https://github.com/apache/flink/pull/6087#discussion_r191165652
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/YarnResourceManager.java ---
    @@ -510,4 +509,39 @@ private int generatePriority(ResourceProfile 
resourceProfile) {
                }
        }
     
    +   /**
    +    * Implements NMClientAsyncCallbackHandler.
    +    */
    +   private class NMClientAsyncCallbackHandler implements 
NMClientAsync.CallbackHandler {
    +
    +           @Override
    +           public void onContainerStarted(ContainerId containerId, 
Map<String, ByteBuffer> allServiceResponse) {
    +
    +           }
    +
    +           @Override
    +           public void onContainerStatusReceived(ContainerId containerId, 
ContainerStatus containerStatus) {
    +
    +           }
    +
    +           @Override
    +           public void onContainerStopped(ContainerId containerId) {
    +
    --- End diff --
    
    Okay. Will change.


---