----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56206/#review164209 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On Feb. 4, 2017, 12:09 a.m., Suma Shivaprasad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56206/ > ----------------------------------------------------------- > > (Updated Feb. 4, 2017, 12:09 a.m.) > > > Review request for atlas. > > > Bugs: ATLAS-1522 > https://issues.apache.org/jira/browse/ATLAS-1522 > > > Repository: atlas > > > Description > ------- > > Currently child(composite) entities are allowed as attribute values in create > or update APIs. Allowing this to have only AtlasObjectId would avoid cycles > in the entity structure > > > Diffs > ----- > > client/src/main/java/org/apache/atlas/AtlasEntitiesClientV2.java 8d1bfa7 > intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 0fb16c6 > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 9494fe4 > > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityWithAssociations.java > 932a40d > intg/src/main/java/org/apache/atlas/model/instance/AtlasObjectId.java > 738f22f > > intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java > c9b6e97 > intg/src/main/java/org/apache/atlas/type/AtlasEntityType.java fcd483c > intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 8bdbe93 > intg/src/test/java/org/apache/atlas/TestUtilsV2.java 0756937 > > repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java > c42f95f > > repository/src/main/java/org/apache/atlas/repository/store/graph/EntityGraphDiscoveryContext.java > 2d748da > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityGraphDiscoveryV1.java > 0e1d9e6 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java > 4c79cef > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java > a989f76 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java > e534d4f > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityMutationContext.java > f6e5055 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityStream.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/IDBasedEntityResolver.java > d02b5a1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/InMemoryMapEntityStream.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/UniqAttrBasedEntityResolver.java > 76308a0 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java > 546cd0c > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java > e3b4efa > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java > da71c31 > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java > bbfcc59 > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java > 4f22437 > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java > 079f3be > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java > 7368c72 > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java > f390e82 > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java > PRE-CREATION > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java > 382d1ef > > webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java > 4b3dd4c > webapp/src/main/java/org/apache/atlas/web/rest/EntitiesREST.java 5107767 > webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java aa209f9 > webapp/src/test/java/org/apache/atlas/web/adapters/TestEntitiesREST.java > cfe9909 > webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java > ee36fdf > > webapp/src/test/java/org/apache/atlas/web/resources/EntityV2JerseyResourceIT.java > d9bffe9 > > Diff: https://reviews.apache.org/r/56206/diff/ > > > Testing > ------- > > ENtityJerseyResourceIt tests failing . This is pending. > > > Thanks, > > Suma Shivaprasad > >
