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 2021/09/10 09:39:37 UTC

[GitHub] [pulsar] BewareMyPower commented on pull request #11570: [Issue 11496][C++] Allow partitioned producers to start lazily

BewareMyPower commented on pull request #11570:
URL: https://github.com/apache/pulsar/pull/11570#issuecomment-916772971


   @codelipenghui I added the `release/2.8.2` label first.
   
   Without this PR, the test introduced from https://github.com/apache/pulsar/pull/11955 would block forever. It's because for a partitioned producer, calling `send()` after  `shutdown()` or `close()` would block forever. I didn't find why this PR works, but this PR should fix it and has been tested in my local env.
   
   The root cause is when a partitioned producer calls `shutdown()` in client shutdown, it only sets the state and the internal producers are not affected. We can also solve this problem by shutdown all internal producers when a partitioned producer calls `shutdown`.


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