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

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

lin-zhao commented on code in PR #17320:
URL: https://github.com/apache/pulsar/pull/17320#discussion_r958985104


##########
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:
   @merlimat  you are exactly right that broker actually does **not** hold the message. I will make an update.



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