m1a2st opened a new pull request, #17440:
URL: https://github.com/apache/kafka/pull/17440

   Jira: https://issues.apache.org/jira/browse/KAFKA-17696
   
   When API calls that handle background events (e.g., poll, unsubscribe, 
close) encounter errors, the errors are only passed to the application thread 
via ErrorEvent.
   Other API calls that do not process background events (e.g., position) are 
not notified of these errors, meaning that issues like unauthorized access to 
topics will go unnoticed by those operations.
   Background operations are not aborted or notified when a metadata error 
occurs, such as an Unauthorized error, which can lead to situations where a 
call like position keeps waiting for an update, despite the Unauthorized error 
already happening.
   
   Due to the blocking issue in 
`applicationEventHandler.addAndGet(checkAndUpdatePositionsEvent);`, I consider 
that we should use `processBackgroundEvents` to get the events, that is better 
than `addAndGet`.
   
   
   ### 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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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

Reply via email to