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

John Zhuge commented on HDFS-9573:
----------------------------------

Hi [~szetszwo],

Thanks for looking at this jira. hashCode and equals should both use "type" 
field or neither of them should use "type" field.

Thanks,
John

> o.a.h.hdfs.protocol.SnapshotDiffReport$DiffReportEntry$hashCode inconsistent 
> with equals
> ----------------------------------------------------------------------------------------
>
>                 Key: HDFS-9573
>                 URL: https://issues.apache.org/jira/browse/HDFS-9573
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: John Zhuge
>            Assignee: John Zhuge
>            Priority: Minor
>         Attachments: HDFS-9573-001.patch
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> DiffReportEntry.equals() uses field "type", but DiffReportEntry.hashCode() 
> doesn't. This breaks the rules on equals and hashCode:
> * if a class overrides equals, it must override hashCode
> * when they are both overridden, equals and hashCode must use the same set of 
> fields



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to