[ 
https://issues.apache.org/jira/browse/FLINK-5856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15894406#comment-15894406
 ] 

ASF GitHub Bot commented on FLINK-5856:
---------------------------------------

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

    https://github.com/apache/flink/pull/3398#discussion_r104148014
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/YarnResourceManager.java ---
    @@ -100,11 +100,15 @@
        /** Client to communicate with the Node manager and launch TaskExecutor 
processes */
        private NMClient nodeManagerClient;
     
    -   /** The number of containers requested, but not yet granted */
    -   private int numPendingContainerRequests;
    +   /** The number of containers requested for each priority, but not yet 
granted */
    +   private Map<Integer, Integer> numPendingContainerRequests = new 
HashMap<>();
    --- End diff --
    
    when creating collections you should always specify an initial capacity.


> Need return redundant containers to yarn for yarn mode
> ------------------------------------------------------
>
>                 Key: FLINK-5856
>                 URL: https://issues.apache.org/jira/browse/FLINK-5856
>             Project: Flink
>          Issue Type: Bug
>          Components: YARN
>            Reporter: shuai.xu
>            Assignee: shuai.xu
>              Labels: flip-6
>
> In flip6, for flink on yarn mode, RM requests container from yarn according 
> to the requirement of the JM. But the AMRMClientAsync used in yarn doesn't 
> guarantee that the number of containers returned exactly equal to the number 
> requested. So it need to record the number request by flink rm and return the 
> redundant ones to yarn.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to