----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69305/ -----------------------------------------------------------
(Updated Nov. 13, 2018, 6:01 a.m.) Review request for atlas, Kapildeo Nayak, Madhan Neethiraj, Nikhil Bonte, Nixon Rodrigues, and Sarath Subramanian. Changes ------- Updates include: Pre-commit build details. Bugs: ATLAS-2774 https://issues.apache.org/jira/browse/ATLAS-2774 Repository: atlas Description ------- **Approach** - New: _DeleteHandlerDelegate_. This is a decorator over _DeleteHandlerV1_. In its implementation it encapsulates: - _SoftDeleteHandler_: Existing implementation. - _HardDeleteHandler_: Existing implementation. - _DefaultHandler_ which is initialized via _atlas-application.properties_. - Modified: _EntityREST_ to accept additional parameter to specify if the delete is hard or soft. - Modified: _AtlasEntityStoreV2_ to accept additional parameter. Existing APIs delegate their function to the new more generic method. - Modified: _ConditionalOnAtlasProperty_ to be applied on field instead of class. This initializes the handler from _atlas-application.properties_. **CURL** Hard ``` curl -X DELETE 'http://localhost:21000/api/atlas/v2/entity/guid/abc8c380-2bfe-47aa-b405-5b79ea9a2a0d?hard=true' ``` Soft ``` curl -X DELETE 'http://localhost:21000/api/atlas/v2/entity/guid/abc8c380-2bfe-47aa-b405-5b79ea9a2a0d?hard=false' ``` Default: As initialized by application based on _atlas-application.properties_ ``` curl -X DELETE 'http://localhost:21000/api/atlas/v2/entity/guid/abc8c380-2bfe-47aa-b405-5b79ea9a2a0d' ``` Diffs ----- intg/src/main/java/org/apache/atlas/store/DeleteType.java PRE-CREATION repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java 750fa1775 repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerDelegate.java PRE-CREATION repository/src/main/java/org/apache/atlas/repository/store/graph/v1/SoftDeleteHandlerV1.java 0bcf0d6cb repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2.java 733369692 repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasRelationshipStoreV2.java 21617dcc3 repository/src/test/java/org/apache/atlas/repository/impexp/ExportSkipLineageTest.java 3393b82d8 repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2Test.java 4fd28206c repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityTestBase.java ba0551050 repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasRelationshipStoreV2Test.java cd1d72787 webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java 00b29e6c8 webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java 68c132c37 webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java 6739364ac webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityRESTDelete.java PRE-CREATION Diff: https://reviews.apache.org/r/69305/diff/4/ Testing (updated) ------- **Unit tests** Additional unit tests. **Functional tests** - Via REST APIs. (See above.) - Tested with atlas.DeleteHandlerV1.impl=org.apache.atlas.repository.store.graph.v1.SoftDeleteHandlerV1 - Tested with atlas.DeleteHandlerV1.impl=org.apache.atlas.repository.store.graph.v1.HardDeleteHandlerV1 **Pre-commit** https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/799/ Thanks, Ashutosh Mestry