[GitHub] [pulsar] merlimat commented on a diff in pull request #17320: [fix][doc] Add more information for producer_request_hold policy

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

merlimat commented on code in PR #17320:
URL: https://github.com/apache/pulsar/pull/17320#discussion_r958635465


##########
site2/docs/admin-api-namespaces.md:
##########
@@ -288,7 +288,7 @@ admin.namespaces().getNamespaceReplicationClusters(namespace)
 
 Backlog quota helps the broker to restrict bandwidth/storage of a namespace once it reaches a certain threshold limit. Admin can set the limit and take corresponding action after the limit is reached.
 
-  1.  producer_request_hold: broker holds but not persists produce request payload
+  1.  producer_request_hold: broker holds but not persists produce request payload until client configuration `operationTimeoutMs` is exceeded.

Review Comment:
   This is not exactly correct. First, the broker does not hold the payload: it's the client that will keep retrying up to the `sendTimeout` configured in the producer. 



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