[jira] [Commented] (MAPREDUCE-4582) [MAPREDUCE-3902] ScheduledRequests#remove should remove the elements from mapsHostMapping and mapsRackMapping

["Siddharth Seth (JIRA)" <ji...@apache.org>]

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

Siddharth Seth commented on MAPREDUCE-4582:
-------------------------------------------

bq. removing entries from attemptToLaunchRequestMap needs to happen after the ScheduledRequests.remove call.
handleTaStopRequest removes the task attempt from attemptToLaunchRequestMap before it attempts to remove the attempt from ScheduledRequests. By the previous comment, I meant this order of removal needs to be fixed.
                
> [MAPREDUCE-3902] ScheduledRequests#remove should remove the elements from mapsHostMapping and mapsRackMapping
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-4582
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4582
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>    Affects Versions: MR-3902
>            Reporter: Tsuyoshi OZAWA
>            Assignee: Tsuyoshi OZAWA
>            Priority: Minor
>         Attachments: MAPREDUCE-4582.2.patch, MAPREDUCE-4582.3.patch, MAPREDUCE-4582.patch
>
>
> ScheduledRequests#remove only remove the specified TaskAttemptId from "maps".
> It's inefficient, and the method should remove the elements from mapsHostMapping and mapsRackMapping.

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