showuon commented on pull request #10794:
URL: https://github.com/apache/kafka/pull/10794#issuecomment-864343641


   @hachikuji , thanks for your explanation. 
   > The client is not necessarily trying to reach the controller. It is 
sending a request to a broker and the fact that we forward it to a controller 
is an implementation detail. So the NOT_CONTROLLER error does not make sense 
for the client. That is why we do not return it. 
   
   >   The intent was to return NOT_CONTROLLER as the error code in the 
EnvelopeResponse itself. Perhaps there is a race condition in the controller 
handling where this is not working? Probably that is what we should fix.
   
   That makes sense! I'll look into it and let you know. Thank you.


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