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

Konstantin Shvachko commented on MAPREDUCE-4342:
------------------------------------------------

Mayank, the patch is not applying as is. Namely the empty line change in 
TrackerDistributedCacheManager. You can just leave the line there. I did that, 
but then it is not compiling. You need to sync it with the repo.

- Could you also change "is been" to "has been" as Robert suggested.
- And add spaces between method parameters.
- Reporting the results of test-patch and test builds would very useful, since 
we don't have Jenkins to verify that for 0.22.

The fix looks good modular the jiras you opened.

                
> Distributed Cache gives inconsistent result if cache files get deleted from 
> task tracker 
> -----------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-4342
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4342
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>    Affects Versions: 0.22.0, 1.0.3, trunk
>            Reporter: Mayank Bansal
>            Assignee: Mayank Bansal
>         Attachments: MAPREDUCE-4342-22-1.patch, MAPREDUCE-4342-22.patch
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to