JiayaoS commented on code in PR #22230:
URL: https://github.com/apache/kafka/pull/22230#discussion_r3204703429


##########
clients/src/main/java/org/apache/kafka/server/authorizer/AuthorizableRequestContext.java:
##########
@@ -49,12 +49,15 @@ public interface AuthorizableRequestContext {
     InetAddress clientAddress();
 
     /**
-     * 16-bit API key of the request from the request header. See
-     * https://kafka.apache.org/protocol#protocol_api_keys for request types.
+     * The 16-bit API key ({@code request_api_key}) from the request header. 
See
+     * https://kafka.apache.org/42/design/protocol/#the-messages

Review Comment:
   I've updated the Javadoc, but since RequestHeader.json is a resource file, 
it can't be referenced via Javadoc's {@link} or @see. That's why I kept the 
documentation URL as a fallback. Would you prefer I drop the URL and just 
reference RequestHeader.json as plain text instead?



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