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

Szilard Nemeth commented on YARN-7556:
--------------------------------------

Hi again!

About my last comment: 

I wasn't aware that this line in the constructor of {{LightWeightResource }} 
sets memory/vcores to the provided value correctly, so that comment can be 
disregarded:
{code:java}
initResourceInformations(value, value, types.length);
{code}
According to our offline discussion with [~templedf], I'm taking this over.

My new patch includes these fixes / modifications on top of patch 007: 
- Some formatting fixes in LightWeightResource
- Rename a local variable in ConfigurableResource#getOneHundredPercentArray
- FairSchedulerConfiguration: readability improvements / code cleanup

[~haibochen]: This is ready for review!
If you commit this, please don't forget to mention Daniel's name as I only 
contributed to the latest patch.
Thanks! 

> Fair scheduler configuration should allow resource types in the minResources 
> and maxResources properties
> --------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-7556
>                 URL: https://issues.apache.org/jira/browse/YARN-7556
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: fairscheduler
>    Affects Versions: 3.0.0-beta1
>            Reporter: Daniel Templeton
>            Assignee: Szilard Nemeth
>            Priority: Critical
>         Attachments: YARN-7556.001.patch, YARN-7556.002.patch, 
> YARN-7556.003.patch, YARN-7556.004.patch, YARN-7556.005.patch, 
> YARN-7556.006.patch, YARN-7556.007.patch, YARN-7556.008.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to