TakaHiR07 opened a new pull request, #303:
URL: https://github.com/apache/pulsar-client-cpp/pull/303

   ### Motivation
   
   When sendAsync batch messages and then flush() and close(), found that the 
callback of message is ResultAlreadyClosed but not Ok. 
   
   The root is if there is no batch messages remain in BatchMessageContainer 
when do flush,  the flush callback directly return Ok. Instead, it should 
return Ok until the lastSendOpFuture complete.  
   
   ### Modifications
   
   1. fix the error code.
   2. test two cases of flush batch messages.
   
   ### Verifying this change
   
   - [x] Make sure that the change passes the CI checks.
   
   ### 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