[ https://issues.apache.org/jira/browse/YARN-9124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16898812#comment-16898812 ]
Adam Antal commented on YARN-9124: ---------------------------------- You can find the patches for the other two branches uploaded into this issue. Pending on jenkins... > Resolve contradiction in ResourceUtils: addMandatoryResources / > checkMandatoryResources work differently > -------------------------------------------------------------------------------------------------------- > > Key: YARN-9124 > URL: https://issues.apache.org/jira/browse/YARN-9124 > Project: Hadoop YARN > Issue Type: Improvement > Reporter: Szilard Nemeth > Assignee: Adam Antal > Priority: Minor > Attachments: YARN-9124.branch-3.1.001.patch, > YARN-9124.branch-3.2.001.patch > > > {{ResourceUtils#addMandatoryResources}}: Adds only memory and vcores as > mandatory resources. > {{ResourceUtils#checkMandatoryResources}}: YARN-6620 added some code to this. > This method not only checks memory and vcores, but all the resources referred > in ResourceInformation#MANDATORY_RESOURCES. > I think it would be good to call {{MANDATORY_RESOURCES}} as > {{PREDEFINED_RESOURCES}} or something like that and use a similar name for > {{checkMandatoryResources}}. -- This message was sent by Atlassian JIRA (v7.6.14#76016) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org