eolivelli opened a new pull request #8526:
URL: https://github.com/apache/pulsar/pull/8526


   ### Motivation
   The tests in pulsar-broker are full of memory leaks, especially about thread 
spawned by non closed instances of:
   - PulsarService
   - BrokerService
   - ManagedLedgerFactoryImpl
   - Executors in general
   
   This is the second part of a series of fixes, with this change it looks like 
the suite is able to use at max 1GB of head and there is a huge reduction of 
the number of threads.
   Unfortunately there are still leaks of Bookies and ZooKeeper server 
instances to be fixed
   
   ### Modifications
   - fix the tests
   - use "alwaysRun=true" in order to ensure that resources are called (the 
default is "false" and if some test is skipped resources are not released)
   
   ### Verifying this change
   This change is a trivial rework / code cleanup without any test coverage.


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to