cmccabe commented on code in PR #14290:
URL: https://github.com/apache/kafka/pull/14290#discussion_r1344800706


##########
clients/src/main/resources/common/message/BrokerRegistrationRequest.json:
##########
@@ -53,6 +53,8 @@
     { "name": "Rack", "type": "string", "versions": "0+", "nullableVersions": 
"0+",
       "about": "The rack which this broker is in." },
     { "name": "IsMigratingZkBroker", "type": "bool", "versions": "1+", 
"default": "false",
-      "about": "If the required configurations for ZK migration are present, 
this value is set to true" }
+      "about": "If the required configurations for ZK migration are present, 
this value is set to true" },
+    { "name": "LogDirs", "type":  "[]uuid", "versions":  "2+",
+      "about": "Log directories configured in this broker which are 
available." }

Review Comment:
   Yeah, it's a good question. I'd say leave it out for now since we're not 
using it yet. When we do use it, it will have to be integrated with Raft (so 
that we "know" that voter 2 with ID B is different than voter 2 with ID A, for 
example).
   
   We can always bump the IBP again when we actually use this on controllers 
(and when we do actually use this, it might have to go into FetchRequest? Not 
sure.)



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