jsancio commented on code in PR #20136:
URL: https://github.com/apache/kafka/pull/20136#discussion_r2228776343


##########
docs/ops.html:
##########
@@ -4027,8 +4027,8 @@ <h5 class="anchor-heading"><a 
id="static_versus_dynamic_kraft_quorums" class="an
   When using a static quorum, the configuration file for each broker and 
controller must specify
   the IDs, hostnames, and ports of all controllers in 
<code>controller.quorum.voters</code>.<p>
 
-  In contrast, when using a dynamic quorum, you should set
-  <code>controller.quorum.bootstrap.servers</code> instead. This configuration 
key need not
+  In contrast, when using a dynamic quorum, 
<code>controller.quorum.bootstrap.servers</code>
+  is set instead and <code>controller.quorum.voters</code> must not be set. 
This configuration key need not

Review Comment:
   Thanks for fixing the documentation. Do you mind swapping the order? Let's 
explain how to configure the node for dynamic voters and in the next paragraph 
explain how to configure for static voters.



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