chia7712 commented on a change in pull request #9741:
URL: https://github.com/apache/kafka/pull/9741#discussion_r542217416
##########
File path: clients/src/main/java/org/apache/kafka/common/protocol/ApiKeys.java
##########
@@ -178,15 +175,7 @@
DescribeGroupsResponseData.SCHEMAS),
LIST_GROUPS(16, "ListGroups", ListGroupsRequestData.SCHEMAS,
ListGroupsResponseData.SCHEMAS),
SASL_HANDSHAKE(17, "SaslHandshake", SaslHandshakeRequestData.SCHEMAS,
SaslHandshakeResponseData.SCHEMAS),
- API_VERSIONS(18, "ApiVersions", ApiVersionsRequestData.SCHEMAS,
ApiVersionsResponseData.SCHEMAS) {
- @Override
- public Struct parseResponse(short version, ByteBuffer buffer) {
- // Fallback to version 0 for ApiVersions response. If a client
sends an ApiVersionsRequest
- // using a version higher than that supported by the broker, a
version 0 response is sent
- // to the client indicating UNSUPPORTED_VERSION.
- return parseResponse(version, buffer, (short) 0);
Review comment:
@dajac Nice question. It is related to #7409 and the replacement is
https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/requests/ApiVersionsResponse.java#L80
----------------------------------------------------------------
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
