----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66797/#review201947 -----------------------------------------------------------
webapp/src/main/java/org/apache/atlas/notification/EntityNotificationListenerV2.java Lines 125 (patched) <https://reviews.apache.org/r/66797/#comment283557> Can 'classifications' list contain some directly associated classifications and some propagated? If yes, shouldn't 2 notifications & 2 audits be created? - Madhan Neethiraj On April 25, 2018, 8:40 a.m., Sarath Subramanian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66797/ > ----------------------------------------------------------- > > (Updated April 25, 2018, 8:40 a.m.) > > > Review request for atlas, Apoorv Naik, Ashutosh Mestry, and Madhan Neethiraj. > > > Bugs: ATLAS-2609 > https://issues.apache.org/jira/browse/ATLAS-2609 > > > Repository: atlas > > > Description > ------- > > Currently audit and notifications are not generated for propagated > classification add/update or delete. > > For Audit: > * Adding Propagated Classification => Send 'Propagated Classifications > Added' message to audit repository. > * Removing Propagated Classification => Send 'Propagated Classifications > Deleted' message to audit repository. > * Updating Propagated Classification => Send 'Propagated Classifications > Updated' message to audit repository. > > For Notification: > * Adding Propagated Classification => Send PROPAGATED_CLASSIFICATION_ADD > message type. > * Removing Propagated Classification => Send PROPAGATED_CLASSIFICATION_DELETE > message type. > * Updating Propagated Classification => Send PROPAGATED_CLASSIFICATION_UPDATE > message type. > > > Diffs > ----- > > client/client-v1/src/main/java/org/apache/atlas/EntityAuditEvent.java > 3cf3e212 > dashboardv2/public/js/utils/Enums.js 8c0518bf > intg/src/main/java/org/apache/atlas/listener/EntityChangeListenerV2.java > 70877d22 > intg/src/main/java/org/apache/atlas/model/audit/EntityAuditEventV2.java > 1045c774 > > intg/src/main/java/org/apache/atlas/v1/model/notification/EntityNotificationV2.java > a8dfd235 > > repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListener.java > 1c04eea4 > > repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListenerV2.java > bb510149 > > repository/src/main/java/org/apache/atlas/repository/converters/AtlasInstanceConverter.java > 20147a0c > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityChangeNotifier.java > 9aebde2b > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreV1.java > 51dcc3a5 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/DeleteHandlerV1.java > c0f2fc3d > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java > 4225a80d > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java > 7d5b7fb6 > > repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreV1Test.java > 8fc0327a > server-api/src/main/java/org/apache/atlas/RequestContextV1.java b7c56865 > > server-api/src/main/java/org/apache/atlas/listener/EntityChangeListener.java > 19fae4fd > > tools/atlas-migration-exporter/src/main/java/org/apache/atlas/migration/NoOpNotificationChangeListener.java > 6c2bf220 > > webapp/src/main/java/org/apache/atlas/notification/EntityNotificationListenerV2.java > ac097b60 > > webapp/src/main/java/org/apache/atlas/notification/NotificationEntityChangeListener.java > a3e5949c > > > Diff: https://reviews.apache.org/r/66797/diff/1/ > > > Testing > ------- > > Precommit in progress: > https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/307/console > > Validated audit/notifications manually for propagated classification > add/update/delete > > > Thanks, > > Sarath Subramanian > >
