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/08/24 17:44:31 UTC

[GitHub] [pulsar] rdhabalia commented on pull request #11753: [pulsar-test] enable pooled netty allocator for unit-tests

rdhabalia commented on pull request #11753:
URL: https://github.com/apache/pulsar/pull/11753#issuecomment-904847109


   > I hope that this change won't introduce instability due to other leaks in the tests
   
   I agree. one of the issue with  `-Dpulsar.allocator.pooled=true` is test starts leaking due to test-failure or missed to close/cleanup broker/client then we will see memory failure soon. So, I was thinking to revert back `-Dpulsar.allocator.pooled=false` and keep direct-memory check in if condition but before that this PR is merged now. let me create a PR and we can merge it if that makes sense.


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