kevin-wu24 commented on code in PR #20136:
URL: https://github.com/apache/kafka/pull/20136#discussion_r2217345250
##########
raft/src/main/java/org/apache/kafka/raft/QuorumConfig.java:
##########
@@ -57,6 +57,9 @@ public class QuorumConfig {
public static final String QUORUM_VOTERS_CONFIG = QUORUM_PREFIX + "voters";
public static final String QUORUM_VOTERS_DOC = "Map of id/endpoint
information for " +
"the set of voters in a comma-separated list of
<code>{id}@{host}:{port}</code> entries. " +
+ "This is the old way of defining membership for controller quorums and
should NOT be " +
+ "set if using dynamic quorums. Instead,
controller.quorum.bootstrap.servers should be set," +
+ "and the voter set is determined by the --standalone or
--initial-controllers flags when formatting." +
Review Comment:
It's the dynamic voter set, since we format with
`--standalone/--initial-controllers`.
--
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
