----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46011/#review128270 -----------------------------------------------------------
addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java (line 1219) <https://reviews.apache.org/r/46011/#comment191689> return false is missing repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java (line 90) <https://reviews.apache.org/r/46011/#comment191688> How are we planing to make this configurable? This is configurable at system or type level? repository/src/main/java/org/apache/atlas/repository/graph/DeleteHandler.java (line 147) <https://reviews.apache.org/r/46011/#comment191690> Can we move this edge Label to a function in TYpedInstanceToGraphMapper so that there is one single place for all the CRUD ops. - Suma Shivaprasad On April 11, 2016, 8:55 a.m., Shwetha GS wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46011/ > ----------------------------------------------------------- > > (Updated April 11, 2016, 8:55 a.m.) > > > Review request for atlas. > > > Bugs: ATLAS-622 > https://issues.apache.org/jira/browse/ATLAS-622 > > > Repository: atlas > > > Description > ------- > > Added soft delete which is enabled by default. Hard delete can be configured > using guice. Had to refactor entity deletes. > > Pending: > 1. > GraphBackedMetadataRepositoryDeleteEntitiesTest.testDisconnectUnidirectionalArrayReferenceFromStructAndTraitTypes > is disabled, need to debug > 2. Have to write more tests for soft deletes > 3. hive hook tests - have fixed some tests, 13/40 are still failing > > > Diffs > ----- > > > addons/hdfs-model/src/main/scala/org/apache/atlas/fs/model/FSDataModel.scala > c380a92 > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/HiveMetaStoreBridge.java > 6b348e2 > addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java > 2f0c71f > repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java > ce1bdfb > > repository/src/main/java/org/apache/atlas/repository/graph/DeleteHandler.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepository.java > b94ff5a > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java > e7e8fb9 > repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java > 895f9df > > repository/src/main/java/org/apache/atlas/repository/graph/GraphToTypedInstanceMapper.java > 1d682bb > > repository/src/main/java/org/apache/atlas/repository/graph/HardDeleteHandler.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/graph/SoftDeleteHandler.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/graph/TypedInstanceToGraphMapper.java > 7011fdf > > repository/src/main/java/org/apache/atlas/repository/typestore/GraphBackedTypeStore.java > 71e9028 > repository/src/test/java/org/apache/atlas/BaseHiveRepositoryTest.java > d07f89d > repository/src/test/java/org/apache/atlas/TestUtils.java a3cf929 > > repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java > ea93cbf > > repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepositoryDeleteEntitiesTest.java > 975d530 > > repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepositoryTest.java > b5440c2 > > repository/src/test/java/org/apache/atlas/repository/graph/GraphRepoMapperScaleTest.java > 3f71eb3 > > repository/src/test/java/org/apache/atlas/repository/graph/SoftDeleteHandlerTest.java > PRE-CREATION > > repository/src/test/java/org/apache/atlas/repository/typestore/GraphBackedTypeStoreTest.java > 8ca3132 > > repository/src/test/java/org/apache/atlas/service/DefaultMetadataServiceTest.java > 156eb3d > repository/src/test/scala/org/apache/atlas/query/GremlinTest.scala 0289b1a > repository/src/test/scala/org/apache/atlas/query/GremlinTest2.scala 79ebfae > repository/src/test/scala/org/apache/atlas/query/LineageQueryTest.scala > cd1c424 > server-api/src/main/java/org/apache/atlas/RequestContext.java 943e4b8 > typesystem/src/main/java/org/apache/atlas/typesystem/IInstance.java ffe40a7 > typesystem/src/main/java/org/apache/atlas/typesystem/Referenceable.java > 7857483 > typesystem/src/main/java/org/apache/atlas/typesystem/Struct.java 70deab2 > > typesystem/src/main/java/org/apache/atlas/typesystem/persistence/DownCastStructInstance.java > d3b9a33 > typesystem/src/main/java/org/apache/atlas/typesystem/persistence/Id.java > d742bb7 > > typesystem/src/main/java/org/apache/atlas/typesystem/persistence/ReferenceableInstance.java > 0fa4666 > > typesystem/src/main/java/org/apache/atlas/typesystem/persistence/StructInstance.java > 16c3a24 > typesystem/src/main/java/org/apache/atlas/typesystem/types/DataTypes.java > 55ec91f > > typesystem/src/test/java/org/apache/atlas/typesystem/json/SerializationJavaTest.java > eb1a15a > > webapp/src/test/java/org/apache/atlas/notification/NotificationHookConsumerIT.java > b2e9f91 > > webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java > 6a939d3 > > Diff: https://reviews.apache.org/r/46011/diff/ > > > Testing > ------- > > UTs, ITs > > > Thanks, > > Shwetha GS > >