You are viewing a plain text version of this content. The canonical link for it is here.

Posted to mapreduce-issues@hadoop.apache.org by "Siddharth Seth (JIRA)" <ji...@apache.org> on 2012/08/31 22:44:09 UTC

[jira] [Commented] (MAPREDUCE-4615) [MAPREDUCE-3902] RMContainerAllocator#assign should be split into functions

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

Siddharth Seth commented on MAPREDUCE-4615:
-------------------------------------------

Tsuyoshi, I'd like to defer committing this patch to a later point, and target functional issues before such a change. Have been a little lazy about creating sub-tasks under 3902. Will do that shortly.
                
> [MAPREDUCE-3902] RMContainerAllocator#assign should be split into functions
> ---------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-4615
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4615
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: applicationmaster
>    Affects Versions: MR-3902
>            Reporter: Tsuyoshi OZAWA
>            Assignee: Tsuyoshi OZAWA
>            Priority: Minor
>         Attachments: MAPREDUCE-4615.patch
>
>
> Currently, the RMContainerAllocator#assign functions does a lot of works inside the one function. This is a bit confusing and make the changes difficult, so this ticket is for refactoring the function and split it into some functions.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira