[GitHub] [pulsar] AlvaroStream commented on a diff in pull request #15368: [improve][doc] Deduplication can have effects in performance and is not activated at client level

[GitBox <gi...@apache.org>]

AlvaroStream commented on code in PR #15368:
URL: https://github.com/apache/pulsar/pull/15368#discussion_r861474012


##########
site2/docs/cookbooks-deduplication.md:
##########
@@ -60,7 +62,9 @@ $ bin/pulsar-admin namespaces set-deduplication \
 
 ## Pulsar clients
 
-If you enable message deduplication in Pulsar brokers, you need complete the following tasks for your client producers:
+If you enable message deduplication in Pulsar brokers, namespaces or topics, it is recommended to make the client retry infinitely the messages until it succeed, otherwise is possible to break the ordering guarantee as some request can time out and the application wont know if the request was successfully added to the topic or not. 

Review Comment:
   Thanks for the link. I will review it next. 
   
   



-- 
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: commits-unsubscribe@pulsar.apache.org

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