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

Hadoop QA commented on MAPREDUCE-2965:
--------------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12494560/MR2965_v3.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 14 new or modified tests.

    -1 patch.  The patch command could not apply the patch.

Console output: 
https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/745//console

This message is automatically generated.

> Streamline hashCode(), equals(), compareTo() and toString() for all IDs
> -----------------------------------------------------------------------
>
>                 Key: MAPREDUCE-2965
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-2965
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: mrv2
>    Affects Versions: 0.23.0, 0.24.0
>            Reporter: Vinod Kumar Vavilapalli
>            Assignee: Siddharth Seth
>             Fix For: 0.23.0, 0.24.0
>
>         Attachments: MR2965_v1.patch, MR2965_v2.patch, MR2965_v3.patch
>
>
> MAPREDUCE-2954 moved these methods to the record interfaces from the PB impls 
> for ContainerId, ApplicationId and ApplicationAttemptId. This is good as they 
> don't need to be tied to the implementation.
> We should do the same for all IDs. In fact some of these are missing for IDs 
> like MR AM JobId, TaskId etc.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to