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

   # Description
   Admin#desceibeReplicaLogDirs fails all futures when any error happens, 
regardless of the exception type or the failed directory. Especially, required 
by the issue, CLUSTER_AUTHORIZATION_FAILED should be handled independently.
   
   # Changes
   ## Tests 
   Two tests are added: one for checking whether user can catch the 
CLUSTER_AUTHORIZATION_FAILED by getCause; one for checking whether failure in 
unrelated directory will affect other requests.
   
   ## CLUSTER_AUTHORIZATION_FAILED
   It is handled by an additional if statement now.
   
   ## Single Directory-level Error
   It is hard to map failed directories back to requested topic partition 
replica. If a directory or a broker fails, the response won't contain 
information of any replicas. The new attempt fails replicas that are not 
mentioned in healthy responses, if there is some error codes. It won't break 
the behavior of giving an empty result to unreplied replicas if no error code 
is in response.
   


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