showuon commented on a change in pull request #11616:
URL: https://github.com/apache/kafka/pull/11616#discussion_r772108732



##########
File path: config/kraft/README.md
##########
@@ -93,7 +96,7 @@ 
controller.quorum.voters=1...@controller1.example.com:9093,2...@controller2.example.co
 
 Each broker and each controller must set `controller.quorum.voters`.  Note 
that the node ID supplied in the `controller.quorum.voters` configuration must 
match that supplied to the server.
 So on controller1, node.id must be set to 1, and so forth.  Note that there is 
no requirement for controller IDs to start at 0 or 1.  However, the easiest and 
least confusing way to allocate
-node IDs is probably just to give each server a numeric ID, starting from 0.
+node IDs is probably just to give each server a numeric ID, starting from 0.  
Also note that the node ID is global and can not conflict with other node ID no 
matter what process role is.

Review comment:
       Add a sentence to mention the node ID is global and can't conflict with 
each other no matter what process role is.




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