rishab-atlan opened a new pull request, #516:
URL: https://github.com/apache/atlas/pull/516

   ## What changes were proposed in this pull request?
   Fix NPE in deleteClassificationV2 when tag missing from Cassandra  
   
   Fixes a NullPointerException in deleteClassificationV2() when attempting to 
delete a tag that exists in Elasticsearch but not in Cassandra 
EntityGraphMapper.java:4772-4777 .
   
   Root cause: ES cleanup happens for missing tags but it doesn't prevent 
subsequent code from attempting to convert a null Tag object to 
AtlasClassification, causing NPE at line 4787 EntityGraphMapper.java:4787 .
   
   Fix: Add early return after queuing ES deferred operation when currentTag is 
null. This allows the ES cleanup to execute successfully via 
executeESPostProcessing() while preventing NPE.
   
   Impact: Playbooks using "set-classification" action will now succeed 
gracefully when encountering tags that exist in ES but not in Cassandra, 
restoring data consistency instead of failing with HTTP 500.
   
   ## How was this patch tested?
   
   (Please explain how this patch was tested. Ex: unit tests, manual tests)
   (If this patch involves UI changes, please attach a screenshot; otherwise, 
remove this)


-- 
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]

Reply via email to