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

ASF GitHub Bot commented on YARN-9095:
--------------------------------------

GitHub user vbmudalige opened a pull request:

    https://github.com/apache/hadoop/pull/445

    YARN-9095. Removed Unused field from Resource: NUM_MANDATORY_RESOURCES

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/vbmudalige/hadoop YARN-9095

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/hadoop/pull/445.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #445
    
----
commit a1cf1a6aa1d8ea93b2efef277878b813c4551e6c
Author: Vidura Mudalige <vidura.mudalige@...>
Date:   2018-12-12T06:40:02Z

    YARN-9095. Removed Unused field from Resource: NUM_MANDATORY_RESOURCES

----


> Removed Unused field from Resource: NUM_MANDATORY_RESOURCES
> -----------------------------------------------------------
>
>                 Key: YARN-9095
>                 URL: https://issues.apache.org/jira/browse/YARN-9095
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Szilard Nemeth
>            Priority: Trivial
>              Labels: newbie, newbie++
>
> I suppose this constant remained in the code from historical reasons, but 
> this is not used anymore so it could be removed.
> This field is especially confusing for new readers, as ResourceInformation 
> now has a field named MANDATORY_RESOURCES and this map contains not only 
> memory and vcores but GPU and FPGA as well.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to