Nicklee007 opened a new pull request, #16837:
URL: https://github.com/apache/pulsar/pull/16837

   ### Motivation
   
   Fix client memory limit currentUsage leak in ProducerImpl. When our pulsar 
cluster occur some error,  producer send message fail and we find the 
`currentUsage` always  keep high value like the leaked, and cause the producer 
send rate is slow.
   Follow 2 point only release the message count semaphore, but not release the 
memory limit.
   
   
https://github.com/apache/pulsar/blob/c217b8f559292fd34c6a4fb4b30aab213720d962/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ProducerImpl.java#L2031-L2033
   
   
https://github.com/apache/pulsar/blob/c217b8f559292fd34c6a4fb4b30aab213720d962/pulsar-client/src/main/java/org/apache/pulsar/client/impl/ProducerImpl.java#L2116-L2120
   
   
   ### Modifications
   
   1. add the `MemoryLimitController` release.
   
   
   ### Documentation
   
   - [X] `doc-not-needed` 
   (Please explain why)
    


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to