hachikuji commented on a change in pull request #11135:
URL: https://github.com/apache/kafka/pull/11135#discussion_r677819183



##########
File path: core/src/main/scala/kafka/server/ControllerApis.scala
##########
@@ -151,6 +150,11 @@ class ControllerApis(val requestChannel: RequestChannel,
     handleRaftRequest(request, response => new 
FetchSnapshotResponse(response.asInstanceOf[FetchSnapshotResponseData]))
   }
 
+  // This API is not wired in yet as the controller does not implement the
+  // metadata completely.
+  // Leaving the code in to be completed and wired in the future
+  //
+  // See https://issues.apache.org/jira/browse/KAFKA-13143 for details

Review comment:
       My preference would be to remove the handler completely. We can always 
bring it back in the future.




-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to