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


##########
site2/docs/admin-api-namespaces.md:
##########
@@ -288,9 +288,9 @@ 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: the producer holds the message and retries until client configuration `operationTimeoutMs` is exceeded

Review Comment:
   Addressed. Good catch. `operationTimeoutMs` applies for producer creation but `sendTimeoutMs` applies for send requests. 



##########
site2/docs/admin-api-namespaces.md:
##########
@@ -288,9 +288,9 @@ 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: the producer holds the message and retries until client configuration `operationTimeoutMs` is exceeded

Review Comment:
   Addressed.



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