[ 
https://issues.apache.org/jira/browse/HADOOP-5214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12674479#action_12674479
 ] 

Matei Zaharia commented on HADOOP-5214:
---------------------------------------

Hi Rahul,

The fix in the patch looks good. I have a few comments about the style though:
- Can you put a space after the commas in the places where you write 
(TaskType.MAP,clusterStatus) and similar?
- There are some tabs in the patch. Change to use all spaces to ensure that we 
have an indent of 2 spaces per level.

> ConcurrentModificationException in FairScheduler.getTotalSlots
> --------------------------------------------------------------
>
>                 Key: HADOOP-5214
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5214
>             Project: Hadoop Core
>          Issue Type: Bug
>          Components: contrib/fair-share
>            Reporter: Vinod K V
>            Assignee: rahul k singh
>         Attachments: 5214.patch
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to