You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/08/08 07:41:28 UTC

[GitHub] [pulsar] Nicklee007 opened a new pull request, #16985: [fix][client]Fix client memory limit currentUsage leak and semaphore release duplicated in ProducerImpl

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

   ### Motivation
   
   This PR is part of work of cherry-pick https://github.com/apache/pulsar/pull/16837 to branch 2.8
   Release memory limit currentUsage in failPendingBatchMessages and avoid duplicate semaphore release in batchMessageAndSend.
   
   ### Modifications
   
   1. Release memory limit currentUsage in failPendingBatchMessages ;
   2. delete duplicate semaphore release in batchMessageAndSend;
   3. fix unit test;
   
   
   ### 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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] gaozhangmin merged pull request #16985: [Branch-2.8][fix][client]Fix client memory limit currentUsage leak and semaphore release duplicated in ProducerImpl

Posted by GitBox <gi...@apache.org>.

gaozhangmin merged PR #16985:
URL: https://github.com/apache/pulsar/pull/16985


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