----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72270/#review220263 -----------------------------------------------------------
intg/src/main/java/org/apache/atlas/listener/EntityChangeListenerV2.java Lines 185 (patched) <https://reviews.apache.org/r/72270/#comment308604> I suggest to have input as list of businessMetadataAttributes. That way events can be put for all businessMetadataAttributes at one go instead putting for 1 attribute at a time. Same may be done for case update, delete. - Nikhil Bonte On April 8, 2020, 2:30 p.m., Mandar Ambawane wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72270/ > ----------------------------------------------------------- > > (Updated April 8, 2020, 2:30 p.m.) > > > Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, > Nixon Rodrigues, Sarath Subramanian, and Sidharth Mishra. > > > Bugs: ATLAS-3689 > https://issues.apache.org/jira/browse/ATLAS-3689 > > > Repository: atlas > > > Description > ------- > > Add audits entries when Namespace Attributes are added/updated/deleted to an > entity > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/listener/EntityChangeListenerV2.java > 2394a12 > intg/src/main/java/org/apache/atlas/model/audit/EntityAuditEventV2.java > 9301e21 > > repository/src/main/java/org/apache/atlas/repository/audit/AtlasAuditService.java > 590f7a0 > > repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListenerV2.java > cab4e1e > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityChangeNotifier.java > 00c0114 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java > 0d5b631 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/IAtlasEntityChangeNotifier.java > c4dc5a1 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/bulkimport/EntityChangeNotifierNop.java > 2943ea9 > > webapp/src/main/java/org/apache/atlas/notification/EntityNotificationListenerV2.java > 6d64fec > > > Diff: https://reviews.apache.org/r/72270/diff/3/ > > > Testing > ------- > > Pre-commit: > https://builds.apache.org/job/PreCommit-ATLAS-Build-Test/1759/console > > Basic testing is done. > > > Thanks, > > Mandar Ambawane > >
