[ https://issues.apache.org/jira/browse/FLINK-5791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15880759#comment-15880759 ]
ASF GitHub Bot commented on FLINK-5791: --------------------------------------- Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/3304#discussion_r102682612 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/taskexecutor/TaskManagerServicesConfiguration.java --- @@ -353,5 +376,16 @@ private static void checkConfigParameter(boolean condition, Object parameter, St name + " : " + parameter + " - " + errorMessage); } } + + private void addResourceProfile(ResourceProfile resourceProfile) { + this.resourceProfiles.add(resourceProfile); + } --- End diff -- imo the configuration object should be immutable. That way one cannot introduce side effects after the object has been created. > Resource should be strictly matched when allocating for yarn > ------------------------------------------------------------ > > Key: FLINK-5791 > URL: https://issues.apache.org/jira/browse/FLINK-5791 > Project: Flink > Issue Type: Improvement > Components: YARN > Reporter: shuai.xu > Assignee: shuai.xu > Labels: flip-6 > > In flip6, for yarn mode, resource should be assigned as requested to avoid > resource wasting and OOM. > 1. YarnResourceManager will request container according to ResourceProfile > in slot request form JM. > 2. RM will pass the ResourceProfile to TM for initializing its slots. > 3. RM should match the slots offered by TM with SlotRequest from JM strictly. -- This message was sent by Atlassian JIRA (v6.3.15#6346)