mjsax commented on code in PR #19973:
URL: https://github.com/apache/kafka/pull/19973#discussion_r2150915147
##########
clients/src/main/java/org/apache/kafka/common/config/TopicConfig.java:
##########
@@ -169,10 +169,13 @@ public class TopicConfig {
"to trigger the unclean leader election immediately if needed.</p>";
public static final String MIN_IN_SYNC_REPLICAS_CONFIG =
"min.insync.replicas";
- public static final String MIN_IN_SYNC_REPLICAS_DOC = "When a producer
sets acks to \"all\" (or \"-1\"), " +
- "this configuration specifies the minimum number of replicas that must
acknowledge " +
- "a write for the write to be considered successful. If this minimum
cannot be met, " +
- "then the producer will raise an exception (either
<code>NotEnoughReplicas</code> or
<code>NotEnoughReplicasAfterAppend</code>).<br> " +
+ public static final String MIN_IN_SYNC_REPLICAS_DOC = "Specifies the
<i>minimum</i> number of in-sync replicas (including the leader) " +
+ "required for a write to be attempted when a producer sets
<code>acks</code> to \"all\" (or \"-1\"). In the <code>acks=all</code> " +
Review Comment:
`attempted` ? Should it be `successful` ?
--
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
