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/07/29 03:50:36 UTC

[GitHub] [pulsar] Nicklee007 commented on pull request #16837: [fix][client]Fix client memory limit currentUsage leak and semaphore release duplicated in ProducerImpl

Nicklee007 commented on PR #16837:
URL: https://github.com/apache/pulsar/pull/16837#issuecomment-1198851455

   @codelipenghui @Jason918 @Shoothzj  Also find producer semaphore release duplicated in `batchMessageAndSend`, if some error cause in  `batchMessageAndSend`, the `semaphoreRelease` will be invoked in `failPendingBatchMessages` .
   Fixed it and add some unit test to cover it.


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