abbccdda commented on a change in pull request #9103:
URL: https://github.com/apache/kafka/pull/9103#discussion_r505842904



##########
File path: clients/src/main/java/org/apache/kafka/common/protocol/Errors.java
##########
@@ -338,7 +339,9 @@
     INCONSISTENT_VOTER_SET(94, "Indicates that the either the sender or 
recipient of a " +
             "voter-only request is not one of the expected voters", 
InconsistentVoterSetException::new),
     INVALID_UPDATE_VERSION(95, "The given update version was invalid.", 
InvalidUpdateVersionException::new),
-    FEATURE_UPDATE_FAILED(96, "Unable to update finalized features due to an 
unexpected server error.", FeatureUpdateFailedException::new);
+    FEATURE_UPDATE_FAILED(96, "Unable to update finalized features due to an 
unexpected server error.", FeatureUpdateFailedException::new),
+    BROKER_AUTHORIZATION_FAILURE(97, "Authorization failed for the request 
during forwarding. " +

Review comment:
       `CLUSTER_AUTHORIZATION_FAILURE` normally indicates a client side 
security configuration error. We intentionally define a separate error code to 
let admin know that there is some security config trouble with the brokers, not 
the clients.




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