zhaijack opened a new pull request #1538: Add rate limit for client lookup 
request
URL: https://github.com/apache/incubator-pulsar/pull/1538
 
 
   ### Motivation
   
   Currently broker side has config `maxConcurrentLookupRequest`, and client 
side has config `concurrentLookupRequest `.  If client side send too much 
lookup request, broker will fail the lookup request with 
`ServerError.TooManyRequests`. This change try to make rate limit of lookup 
request from client side.
   
   ### Modifications
   
   Add new parameter `maxLookupRequest` in clientConfiguration, and do rate 
limit in ClientCnx.
   
   ### Result
   
   Client side lookup could be rate-limited, and avoid server side return error.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

Reply via email to