----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71447/#review217629 -----------------------------------------------------------
repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java Lines 326 (patched) <https://reviews.apache.org/r/71447/#comment304921> Using Integer, instead of String, can result in better performance. Value '0' be treated as 'false'; and other values as 'true'. Please review. repository/src/main/java/org/apache/atlas/repository/store/graph/v2/UniqAttrBasedEntityResolver.java Line 41 (original), 45 (patched) <https://reviews.apache.org/r/71447/#comment304920> Instead of referencing configuration directly here (i.e. #91), consider using: RequestContext.get().isCreateDummyEntityForNonExistingReference()); And have populate RequestContext from: - NotificationHookConsumer.handleMessage() - AuditFilter.doFilter(), similar to call RequestContext.get().setDeleteType() - Madhan Neethiraj On Sept. 7, 2019, 7:06 a.m., Sarath Subramanian wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71447/ > ----------------------------------------------------------- > > (Updated Sept. 7, 2019, 7:06 a.m.) > > > Review request for atlas, Sridhar K and Madhan Neethiraj. > > > Bugs: ATLAS-3405 > https://issues.apache.org/jira/browse/ATLAS-3405 > > > Repository: atlas > > > Description > ------- > > Hook notifications containing references to non-existing entities are dropped > by Atlas server, with the following error: > > AtlasObjectId{guid='null', typeName='hive_db', > uniqueAttributes={qualifiedName:db1@prod}} is not found > Instead of dropping such notifications, it will help to create an entity for > the reference, and complete processing of the notifications. Such entities > will only have its unique-attribute populated, however will capture the > lineage and other relationships. When Atlas receives details of the entity in > a subsequent notification, Atlas will have rest of the attributes populated > as well. > > > Diffs > ----- > > common/src/main/java/org/apache/atlas/repository/Constants.java 673a85cb9 > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java > 659f7d586 > intg/src/main/java/org/apache/atlas/model/instance/AtlasEntityHeader.java > 26687bffc > > repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java > a1776c843 > repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java > a58c67059 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityGraphDiscoveryV2.java > 44df3512e > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2.java > c495dcc7a > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java > 8ddbf748c > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphRetriever.java > 02638c3ba > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/IDBasedEntityResolver.java > fe76b3a89 > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/UniqAttrBasedEntityResolver.java > d400a1df9 > > webapp/src/main/java/org/apache/atlas/notification/EntityNotificationListenerV2.java > a844bcdf6 > > > Diff: https://reviews.apache.org/r/71447/diff/1/ > > > Testing > ------- > > Precommit: > https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/1384/console > > > Thanks, > > Sarath Subramanian > >
