achandel-01 commented on code in PR #768:
URL: https://github.com/apache/atlas/pull/768#discussion_r4192071709
##########
webapp/src/main/java/org/apache/atlas/notification/SerialEntityProcessor.java:
##########
@@ -575,6 +584,10 @@ public TopicPartitionOffsetResult handleMessage(Ticket
ticket) {
final AtlasObjectId entityId
= partialUpdateRequest.getEntityId();
final AtlasEntityWithExtInfo entity
= partialUpdateRequest.getEntity();
+ if (isIgnoredByConfiguredPattern(entity != null ?
entity.getEntity() : entityId, kafkaMsg)) {
Review Comment:
For ENTITY_PARTIAL_UPDATE_V2, run ignore on entityId, not on
entity.getEntity() alone. We hit this on ALTER TABLE … SET TBLPROPERTIES after
a CREATE that was already ignored ,the patch often has only parameters and no
qualifiedName, so patch-only ignore did not run, updateEntity was called, and
we got INSTANCE_BY_UNIQUE_ATTRIBUTE_NOT_FOUND. entityId carries the target type
and QN — check that first; the patch is optional.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
