xakassi opened a new pull request #9211:
URL: https://github.com/apache/kafka/pull/9211


   DistributedHerder goes to updateConfigsWithIncrementalCooperative() 
synchronized method and called configBackingStore.snapshot() which take a lock 
on internal object in KafkaConfigBackingStore class.
   
   Meanwhile KafkaConfigBackingStore in ConsumeCallback inside synchronized 
block on internal object gets SESSION_KEY record and calls 
updateListener.onSessionKeyUpdate() which take a lock on DistributedHerder.
   
   So, we have a Deadlock.
   
   To avoid this, updateListener with new session key should be called outside 
synchronized block as it's done, for example, for 
updateListener.onTaskConfigUpdate(updatedTasks).
   
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to