atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Shwetha GS <sshivalingamur...@hortonworks.com>
Subject Review Request 46011: ATLAS-622 Introduce soft delete
Date Mon, 11 Apr 2016 08:56:00 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46011/
-----------------------------------------------------------

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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message