ableegoldman commented on a change in pull request #10690:
URL: https://github.com/apache/kafka/pull/10690#discussion_r632788590



##########
File path: 
clients/src/main/java/org/apache/kafka/clients/producer/ProducerConfig.java
##########
@@ -247,9 +248,10 @@
     public static final String ENABLE_IDEMPOTENCE_DOC = "When set to 'true', 
the producer will ensure that exactly one copy of each message is written in 
the stream. If 'false', producer "
                                                         + "retries due to 
broker failures, etc., may write duplicates of the retried message in the 
stream. "
                                                         + "Note that enabling 
idempotence requires <code>" + MAX_IN_FLIGHT_REQUESTS_PER_CONNECTION + "</code> 
to be less than or equal to 5, "
-                                                        + "<code>" + 
RETRIES_CONFIG + "</code> to be greater than 0 and <code>" + ACKS_CONFIG + 
"</code> must be 'all'. If these values "
+                                                        + "<code>" + 
RETRIES_CONFIG + "</code> to be greater than 0, and <code>" + ACKS_CONFIG + 
"</code> must be 'all'. If these values "
                                                         + "are not explicitly 
set by the user, suitable values will be chosen. If incompatible values are 
set, "
-                                                        + "a 
<code>ConfigException</code> will be thrown.";
+                                                        + "a 
<code>ConfigException</code> will be thrown. With an idempotent producer, 
setting the <code>" + MAX_IN_FLIGHT_REQUESTS_PER_CONNECTION
+                                                        + "</code> greater 
than 1 will not break ordering guarantees.";

Review comment:
       will do




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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to