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

Daniel Templeton commented on YARN-7316:
----------------------------------------

I've actually been actively doing the opposite.  It seems strange to me that we 
should pass the calc object to {{Resources}} so that it can call the method for 
us.  It's harder to read, makes for longer lines that may need to be wrapped, 
and creates unnecessary coupling.  It's not like using {{Resources}} as a proxy 
saves you from needing a calculator instance or does anything useful for you at 
all, really.

What's the motivation for running all the calls through the {{Resources}} 
object?  It makes sense for some of the comparisons, where it's actually doing 
something useful, but I don't get it for the rest of the calls.

> Cleaning up the usage of Resources and ResourceCalculator
> ---------------------------------------------------------
>
>                 Key: YARN-7316
>                 URL: https://issues.apache.org/jira/browse/YARN-7316
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>    Affects Versions: 3.1.0
>            Reporter: lovekesh bansal
>            Assignee: lovekesh bansal
>            Priority: Minor
>             Fix For: 3.1.0
>
>         Attachments: YARN-7316_trunk.001.patch
>
>
> Cleaning up and Uniformizing the usage the usage of Resources and 
> ResourceCalculator.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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