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

Andras Gyori commented on YARN-10503:
-------------------------------------

Thank you [~zhuqi] for the patch. I think I understand the points made by 
[~ebadger], but I see that the repetition was necessary, because 
AbsoluteResourceType.valueOf would throw an exception if encountering an 
unknown secondary resource. I thought about a solution, but I do not think it 
is worth it.

My only suggestion would be to move getCustomResourcesStrings to the 
ResourceUtils instead. Apart from this, it looks good to me.

 

> Support queue capacity in terms of absolute resources with custom 
> resourceType.
> -------------------------------------------------------------------------------
>
>                 Key: YARN-10503
>                 URL: https://issues.apache.org/jira/browse/YARN-10503
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Qi Zhu
>            Assignee: Qi Zhu
>            Priority: Critical
>         Attachments: YARN-10503.001.patch, YARN-10503.002.patch, 
> YARN-10503.003.patch, YARN-10503.004.patch, YARN-10503.005.patch, 
> YARN-10503.006.patch, YARN-10503.007.patch, YARN-10503.008.patch, 
> YARN-10503.009.patch
>
>
> Now the absolute resources are memory and cores.
> {code:java}
> /**
>  * Different resource types supported.
>  */
> public enum AbsoluteResourceType {
>   MEMORY, VCORES;
> }{code}
> But in our GPU production clusters, we need to support more resourceTypes.
> It's very import for cluster scaling when with different resourceType 
> absolute demands.
>  
> This Jira will handle GPU first.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
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