erobot opened a new pull request, #392:
URL: https://github.com/apache/pulsar-client-cpp/pull/392
### Motivation
Current code releases the producer's pending semaphore twice when batch is
on and message is too big. The unbalanced release overflows the semaphore, and
subsequent sends will fail with ProducerQueueIsFull.
### Modifications
Remove the redundant semaphore release as the necessary release will be done
in `handleFailedResult`.
### Verifying this change
- [ ] Make sure that the change passes the CI checks.
This change modified a existing test and can be verified as follows:
- ProducerTest.testMaxMessageSize
### Documentation
<!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
- [ ] `doc-required`
(Your PR needs to update docs and you will update later)
- [x] `doc-not-needed`
Bug fix.
- [ ] `doc`
(Your PR contains doc changes)
- [ ] `doc-complete`
(Docs have been already added)
--
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]
