rhauch opened a new pull request #8536:
URL: https://github.com/apache/kafka/pull/8536


   When the Connect worker forwards a REST API request to the leader, it might 
get back a `RequestTargetException` that suggests the worker should forward the 
request to a different worker. This can happen when the leader changes, and the 
worker that receives the original request forwards the request to the worker 
that it thinks is the current leader, but that worker is not the current 
leader. In this case. In most cases, the worker that received the forwarded 
request includes the URL of the current leader, but it is possible (albeit 
rare) that the worker doesn’t know the current leader and will include a null 
leader URL in the resulting `RequestTargetException`.
   
   When this rare case happens, the user gets a null pointer exception in their 
response and the NPE is logged. Instead, the worker should catch this condition 
and provide a more useful error message that is similar to other existing error 
messages that might occur.
   
   Added a unit test that verifies this corner case is caught and this 
particular NPE does not occur.
   
   ### 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