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

(Updated April 15, 2016, 4:46 a.m.)


Review request for atlas.


Changes
-------

Removed some un-necessary changes that were there


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 (updated)
-----

  
addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/HiveMetaStoreBridge.java
 f007a32 
  addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java 
a0f1d40 
  addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java 
dbba926 
  client/src/main/java/org/apache/atlas/AtlasClient.java cc87706 
  common/src/main/java/org/apache/atlas/ApplicationProperties.java ca72ffd 
  repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java 
75d14f0 
  
repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListener.java
 7f77feb 
  
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/GraphBackedRepositoryHardDeleteTest.java
 PRE-CREATION 
  
repository/src/test/java/org/apache/atlas/repository/graph/GraphBackedRepositorySoftDeleteTest.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
 1156b67 
  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 
  repository/src/test/scala/org/apache/atlas/query/QueryTestsUtils.scala 
1a0fa20 
  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/test/java/org/apache/atlas/ApplicationPropertiesTest.java 
d8098ac 
  typesystem/src/test/resources/atlas-application.properties dbd6002 
  
webapp/src/test/java/org/apache/atlas/notification/NotificationHookConsumerIT.java
 b2e9f91 
  
webapp/src/test/java/org/apache/atlas/web/resources/EntityJerseyResourceIT.java 
23f6874 

Diff: https://reviews.apache.org/r/46011/diff/


Testing
-------

UTs, ITs


Thanks,

Shwetha GS

Reply via email to