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/12/09 12:38:03 UTC

[GitHub] [pulsar] aloyszhang opened a new issue #13214: producer create failed with `connectionFailed ` will leak some resource

aloyszhang opened a new issue #13214:
URL: https://github.com/apache/pulsar/issues/13214


   **Describe the bug**
   As describe in #13055，in the progress of building a new producer,  if `connectionFailed ` happens, there will left some resources not released like the `sendTimeout` TimerTask.
   
   **To Reproduce**
   Steps to reproduce the behavior:
   1. create a new producer 
   2. simulate a `connectionFailed ` 
   3. check the `client.timer.pendingTimeouts()` , result will > 0
   
   **Expected behavior**
   Once `connectionFailed ` happened, the producer will not available anymore, so all resources should be released.
   
   **Additional context**
   
   when `connectionFailed ` , check the `sendTimeout`, `batchTimerTask`, `keyGeneratorTask`，release them if not null.


-- 
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] codelipenghui closed issue #13214: producer create failed with `connectionFailed ` will leak some resource

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

codelipenghui closed issue #13214:
URL: https://github.com/apache/pulsar/issues/13214


   


-- 
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] aloyszhang commented on issue #13214: producer create failed with `connectionFailed ` will leak some resource

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

aloyszhang commented on issue #13214:
URL: https://github.com/apache/pulsar/issues/13214#issuecomment-989814283


   I'm gonna send a PR soon.


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