----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72060/#review219465 -----------------------------------------------------------
Ship it! Ship It! - Sarath Subramanian On Jan. 30, 2020, 4:10 a.m., Pinal Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72060/ > ----------------------------------------------------------- > > (Updated Jan. 30, 2020, 4:10 a.m.) > > > Review request for atlas, Madhan Neethiraj, Nixon Rodrigues, and Sarath > Subramanian. > > > Bugs: ATLAS-3601 > https://issues.apache.org/jira/browse/ATLAS-3601 > > > Repository: atlas > > > Description > ------- > > 1) When 1 label among multiple labels is removed , LABEL_DELETE is audited. > But when all labels are deleted or the last label is deleted , no > LABEL_DELETE is audited. > 2)When we add label and delete label from multiple label present, > simultaneously, Only LABEL_ADD is audited > > > Diffs > ----- > > > repository/src/main/java/org/apache/atlas/repository/audit/AbstractStorageBasedAuditRepository.java > f6b741b5e > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java > e427a59e7 > > > Diff: https://reviews.apache.org/r/72060/diff/1/ > > > Testing > ------- > > 1) Added one label - one audit for LABEL_ADD > 2) Added multiple labels simultaneously - one audit for LABEL_ADD,with > multiple label names in Details > 3) Deleted one of the label - one audit for LABEL_DELETE > 4) Deleted multiple labels simultaneously - one audit for LABEL_DELETE,with > multiple label names in Details > 5) deleled last label left - one audit for LABEL_DELETE > 6) Added one label and deleted two labels simultaneously -two audits, one > audit for LABEL_ADD and second for LABEL_DELETE ,with multiple label names in > Details > > > Thanks, > > Pinal Shah > >