----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55531/#review162247 -----------------------------------------------------------
intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java (line 99) <https://reviews.apache.org/r/55531/#comment233562> This method is already defined in the base class - AtlasStructType. Consider removing it from here. intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java (line 102) <https://reviews.apache.org/r/55531/#comment233563> This override simply calls the base class impl. Consider removing it from here. intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java (line 129) <https://reviews.apache.org/r/55531/#comment233567> Since the base class has 'allAttributes', can getAttributeDef() be moved to the base class? intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java (line 73) <https://reviews.apache.org/r/55531/#comment233564> is "allAd" needed here? intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java (line 101) <https://reviews.apache.org/r/55531/#comment233565> This method is already defined in the base class - AtlasStructType. Consider removing it from here. intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java (line 104) <https://reviews.apache.org/r/55531/#comment233566> This override simply calls the base class impl. Consider removing from here. intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java (line 126) <https://reviews.apache.org/r/55531/#comment233568> Since the base class has 'allAttributes', can getAttributeDef() be moved to the base class? intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java (line 270) <https://reviews.apache.org/r/55531/#comment233569> 'allAttributeDefs' argument may not be necessary. Please review. intg/src/main/java/org/apache/atlas/type/AtlasStructType.java (line 80) <https://reviews.apache.org/r/55531/#comment233570> Shouldn't this handle 'attributeName' from superTypes as well - by looking into 'allAttributes', instead of 'structDef'. intg/src/main/java/org/apache/atlas/type/AtlasStructType.java (line 516) <https://reviews.apache.org/r/55531/#comment233571> Consider taking only the following 2 arguments, as the rest can be derived from these: AtlasAttribute(AtlasStructType structType, String attributeName) { this.structType = structType; this.structDef = structType.getStructDef(); this.attributeDef = structType.getAttributeDef(attributeName); this.attributeType = structType.getAttributeType(attributeName); this.qualifiedName = getQualifiedAttributeName(structDef, attributeName); } - Madhan Neethiraj On Jan. 19, 2017, 6:22 a.m., Suma Shivaprasad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55531/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2017, 6:22 a.m.) > > > Review request for atlas. > > > Bugs: ATLAS-1467 > https://issues.apache.org/jira/browse/ATLAS-1467 > > > Repository: atlas > > > Description > ------- > > Implemented > > 1. Entity graph discovery - This differs from current implementation in that > it collects all references first and resolves them later . The current > implemenation tries to resolve references at the same time resulting in > issues when entities have cross references to each other . Also it handles > resolving a refernce by its unique attribute (by supplying an AtlasEntity > instead of an Id) > 2. Create with new APIs and POJOs ( no change in graph model ) > 3. Full Update with new APIs and POJOs ( no change in graph model ) > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 6770c41 > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 2ad0f76 > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityHeader.java > e7b70aa > intg/src/main/java/org/apache/atlas/model/instance/EntityMutations.java > 3501c90 > intg/src/main/java/org/apache/atlas/model/typedef/AtlasEntityDef.java > 85e9d77 > intg/src/main/java/org/apache/atlas/model/typedef/AtlasStructDef.java > aee26ef > intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java > 8772720 > intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java 3625f72 > intg/src/main/java/org/apache/atlas/type/AtlasStructType.java e20af76 > intg/src/test/java/org/apache/atlas/TestUtilsV2.java 53b109c > intg/src/test/java/org/apache/atlas/type/TestAtlasTypeRegistry.java 9429c07 > repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java > 54dda50 > > repository/src/main/java/org/apache/atlas/discovery/EntityDiscoveryService.java > 2be9a2d > > repository/src/main/java/org/apache/atlas/discovery/EntityLineageService.java > 45e2dd2 > repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java > 5259249 > > repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java > f17b816 > > repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscovery.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/EntityResolver.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/ArrayVertexMapper.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java > e731c11 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEnumDefStoreV1.java > fccbeba > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasGraphUtilsV1.java > 18b3b85 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasStructDefStoreV1.java > e780dc1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/GraphMutationContext.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/HardDeleteHandlerV1.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InstanceGraphMapper.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/MapVertexMapper.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/SoftDeleteHandlerV1.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/StructVertexMapper.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/util/AtlasRepositoryConfiguration.java > ea0e670 > > repository/src/test/java/org/apache/atlas/lineage/EntityLineageServiceTest.java > b1dac9d > > repository/src/test/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStoreTest.java > c7c3286 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java > PRE-CREATION > server-api/src/main/java/org/apache/atlas/RequestContextV1.java > PRE-CREATION > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java > c4be236 > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java > ad16be7 > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java > 3565ab3 > webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java f6acd07 > webapp/src/main/java/org/apache/atlas/web/util/LineageUtils.java 54ca236 > webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java > 265b650 > webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java > 2a75773 > webapp/src/test/java/org/apache/atlas/web/resources/BaseResourceIT.java > dcb1264 > > webapp/src/test/java/org/apache/atlas/web/resources/EntityDiscoveryJerseyResourceIT.java > 2ade5b0 > > webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java > d7702e2 > > Diff: https://reviews.apache.org/r/55531/diff/ > > > Testing > ------- > > UTs for create > > TODO: > Update testing pending > Error handling for unresolved references > UTS for discovery > > > Thanks, > > Suma Shivaprasad > >
