----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46011/#review128551 -----------------------------------------------------------
repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java (line 90) <https://reviews.apache.org/r/46011/#comment191988> Configuring hard deletes at the system level should be done via the Atlas configuration properties (i.e. atlas-application.properties) or something that can be configured by Atlas consumers - not a compile-time mechanism like Guice. Also, for our use cases, it would be strongly preferable to enable hard delete at the API and/or type level and not be limited to a system-wide configuration. - David Kantor On April 12, 2016, 3:27 p.m., Shwetha GS wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46011/ > ----------------------------------------------------------- > > (Updated April 12, 2016, 3:27 p.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/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/HiveMetaStoreBridge.java > 6b348e2 > addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java > b947a8c > addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java > 2f0c71f > client/src/main/java/org/apache/atlas/AtlasClient.java 938a0a3 > repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java > ce1bdfb > > repository/src/main/java/org/apache/atlas/repository/graph/AtlasEdgeLabel.java > da2ad9a > > 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/GraphBackedMetadataRepositoryDeleteTestBase.java > PRE-CREATION > > repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepositoryTest.java > b5440c2 > > repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedReposiotrySoftDeleteTest.java > PRE-CREATION > > repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedRepositoryHardDeleteTest.java > PRE-CREATION > > repository/src/test/java/org/apache/atlas/repository/graph/GraphRepoMapperScaleTest.java > 3f71eb3 > > 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 > >
