----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69707/#review212127 -----------------------------------------------------------
Fix it, then Ship it! repository/src/main/java/org/apache/atlas/repository/store/graph/v2/ClassificationAssociator.java Lines 60 (patched) <https://reviews.apache.org/r/69707/#comment297750> It might be simpler to add following method in EntityGraphRetriever directly, instead of a separate class ClassificationAssociator. public class EntityGraphRetriever { AtlasEntityHeaders getEntitiesWithTagChanges(fromTimestamp, toTimestamp) { ... } } repository/src/main/java/org/apache/atlas/repository/store/graph/v2/ClassificationAssociator.java Lines 80 (patched) <https://reviews.apache.org/r/69707/#comment297746> 'Updater' class is no more used. If true, please remove. repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphRetriever.java Line 98 (original), 98 (patched) <https://reviews.apache.org/r/69707/#comment297745> Why remove 'final' here? - Madhan Neethiraj On Jan. 16, 2019, 5:40 p.m., Ashutosh Mestry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69707/ > ----------------------------------------------------------- > > (Updated Jan. 16, 2019, 5:40 p.m.) > > > Review request for atlas, Madhan Neethiraj, Nikhil Bonte, Nixon Rodrigues, > and Sarath Subramanian. > > > Bugs: ATLAS-3029 > https://issues.apache.org/jira/browse/ATLAS-3029 > > > Repository: atlas > > > Description > ------- > > **Approach** > - Scan HBase 'ATLAS_ENTITY_AUDIT_EVENTS' table with approprite paramters. > - New: Package the output into _AtlasEntityHeaders_. > - Provide _REST APIs_ to _EntityREST_. > - _AtlasClientV2_ updated to support the new endpoints. > > **Rules for _setClassifications_** > - If incoming _AtlasEntityHeader_: > - Has no classifications: Delete entity's classification. > - Has classifications & entity does not have the classification: Add to > entity's classifications. > - Has classifications & entity has the classification: Update entity's > classification. > > > **CURL** > > _Fetch_ > curl -X GET -u admin:passWord -H 'Content-Type: application/json' > > 'http://localhost:21000/api/atlas/v2/entity/bulk/headers?tagUpdateStartTime=1546992000000' > \ > > _Udpate_ > curl -X POST -u admin:passWord -H 'Content-Type: application/json' > 'http://localhost:21000/api/atlas/v2/entity/bulk/setClassifications' > -d @fetched-classifications.json > > > **Not supported** > - Cassandra based Audits. > - For testing: In-memory audits. > > > Diffs > ----- > > client/client-v2/src/main/java/org/apache/atlas/AtlasClientV2.java > 7c8caee89 > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityHeaders.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/audit/CassandraBasedAuditRepository.java > eb78f8f70 > > repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditRepository.java > aab2d5b6c > > repository/src/main/java/org/apache/atlas/repository/audit/HBaseBasedAuditRepository.java > 6f4415fa8 > > repository/src/main/java/org/apache/atlas/repository/audit/InMemoryEntityAuditRepository.java > dca3b853e > > repository/src/main/java/org/apache/atlas/repository/audit/NoopEntityAuditRepository.java > e3a607809 > > repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasEntityStore.java > 750fa1775 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2.java > 35aa3afed > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/ClassificationAssociator.java > PRE-CREATION > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphRetriever.java > 066abc109 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v2/ClassificationAssociatorTest.java > PRE-CREATION > > repository/src/test/resources/json/classification-association/col-entity-2-classification.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/col-entity-2-classifications.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/col-entity-2.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/entity-header-1.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/entity-header-2-classifications.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/entity-header-2-no-classifications.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/entity-header-2.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/entity-header-3.json > PRE-CREATION > > repository/src/test/resources/json/classification-association/entity-header-empty.json > PRE-CREATION > webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java 68c132c37 > > > Diff: https://reviews.apache.org/r/69707/diff/7/ > > > Testing > ------- > > **Unit tests** > New tests added. > > **Functional tests** > Via CURL calls. > > **Volume tests** > Loads of over 2000 entities. > > > File Attachments > ---------------- > > entity-headers.json > > https://reviews.apache.org/media/uploaded/files/2019/01/14/a23bead7-6990-4d5b-b3b8-f46f990f5d99__entity-headers.json > > > Thanks, > > Ashutosh Mestry > >
