----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53748/#review157267 -----------------------------------------------------------
Please run all tests as well to verify that these changes do not introduce new test failures typesystem/src/main/java/org/apache/atlas/typesystem/Referenceable.java (line 177) <https://reviews.apache.org/r/53748/#comment227791> Seems like equals and hashcode methods have been declared twice typesystem/src/main/java/org/apache/atlas/typesystem/Referenceable.java (line 213) <https://reviews.apache.org/r/53748/#comment227792> Changes are required here as well typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeInfo.java (line 99) <https://reviews.apache.org/r/53748/#comment227793> Duplicate. Please remove - Vimal Sharma On Nov. 28, 2016, 8:32 p.m., Apoorv Naik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53748/ > ----------------------------------------------------------- > > (Updated Nov. 28, 2016, 8:32 p.m.) > > > Review request for atlas, Madhan Neethiraj, Sarath Subramanian, Shwetha GS, > and Suma Shivaprasad. > > > Bugs: ATLAS-1303 > https://issues.apache.org/jira/browse/ATLAS-1303 > > > Repository: atlas > > > Description > ------- > > Updated equals and hashCode methods > > Using JDK7's built-in classes to simplify the code and make it more > readable/understandable. > > > Diffs > ----- > > catalog/src/main/java/org/apache/atlas/catalog/BaseRequest.java b0e428f > client/src/main/java/org/apache/atlas/EntityAuditEvent.java 29a04ab > common/src/main/java/org/apache/atlas/ha/HAConfiguration.java aefaef5 > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 5288cbf > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityHeader.java > d9c74ae > > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java > 4ddd585 > intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java > bedd4d9 > intg/src/main/java/org/apache/atlas/model/instance/AtlasStruct.java 695f1bb > > intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java > 72f4118 > intg/src/main/java/org/apache/atlas/model/instance/EntityMutations.java > e489f33 > intg/src/main/java/org/apache/atlas/model/lineage/AtlasLineageInfo.java > 61b7f91 > intg/src/main/java/org/apache/atlas/model/typedef/AtlasBaseTypeDef.java > feae00d > > intg/src/main/java/org/apache/atlas/model/typedef/AtlasClassificationDef.java > 1be5ee1 > intg/src/main/java/org/apache/atlas/model/typedef/AtlasEntityDef.java > 0312176 > intg/src/main/java/org/apache/atlas/model/typedef/AtlasEnumDef.java f486f9b > intg/src/main/java/org/apache/atlas/model/typedef/AtlasStructDef.java > ef12896 > intg/src/main/java/org/apache/atlas/model/typedef/AtlasTypeDefHeader.java > a3929b3 > > notification/src/main/java/org/apache/atlas/notification/MessageVersion.java > 6ef407a > > notification/src/main/java/org/apache/atlas/notification/entity/EntityNotificationImpl.java > fda588e > > notification/src/test/java/org/apache/atlas/notification/AbstractNotificationConsumerTest.java > ed5b9fc > repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java > cdeb117 > typesystem/src/main/java/org/apache/atlas/typesystem/Referenceable.java > 0b3ffe3 > typesystem/src/main/java/org/apache/atlas/typesystem/persistence/Id.java > a3f9421 > > typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeDefinition.java > f556223 > > typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeInfo.java > 9cb0d0d > > typesystem/src/main/java/org/apache/atlas/typesystem/types/EnumTypeDefinition.java > ac3b5a3 > > typesystem/src/main/java/org/apache/atlas/typesystem/types/HierarchicalTypeDefinition.java > 8069422 > > typesystem/src/main/java/org/apache/atlas/typesystem/types/Multiplicity.java > 18ef2ee > > typesystem/src/main/java/org/apache/atlas/typesystem/types/StructTypeDefinition.java > c9316cc > webapp/src/main/java/org/apache/atlas/web/params/AbstractParam.java 9cfdb76 > > Diff: https://reviews.apache.org/r/53748/diff/ > > > Testing > ------- > > Working on executing all tests > > > Thanks, > > Apoorv Naik > >